Skip to content

Commit

Permalink
Fix use of CreateAlignedStore (#37388)
Browse files Browse the repository at this point in the history
This is a compile error on 10+ and is probably triggering implicity conversion on LLVM 8 and 9.
  • Loading branch information
yuyichao authored Sep 4, 2020
1 parent 0c6e9ef commit 5acd7bd
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/codegen.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -4102,7 +4102,7 @@ static void emit_upsilonnode(jl_codectx_t &ctx, ssize_t phic, jl_value_t *val)
if (!val) {
if (vi.boxroot) {
// memory optimization: eagerly clear this gc-root now
ctx.builder.CreateAlignedStore(V_rnull, vi.boxroot, true, Align(sizeof(void*)));
ctx.builder.CreateAlignedStore(V_rnull, vi.boxroot, Align(sizeof(void*)), true);
}
if (vi.pTIndex) {
// We don't care what the contents of the variable are, but it
Expand All @@ -4111,7 +4111,7 @@ static void emit_upsilonnode(jl_codectx_t &ctx, ssize_t phic, jl_value_t *val)
ctx.builder.CreateAlignedStore(
vi.boxroot ? ConstantInt::get(T_int8, 0x80) :
ConstantInt::get(T_int8, 0x01),
vi.pTIndex, true, Align(1));
vi.pTIndex, Align(1), true);
}
else if (vi.value.V && !vi.value.constant && vi.value.typ != jl_bottom_type) {
assert(vi.value.ispointer());
Expand Down

0 comments on commit 5acd7bd

Please sign in to comment.