[llvm] [Attributor] Fix Load/Store offsets in case of multiple access bins when an allocation size is changed. (PR #72029)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 5 12:51:45 PST 2024


================
@@ -22,18 +22,18 @@ define i32 @f(i32 %x) {
 ; TUNIT-LABEL: define {{[^@]+}}@f
 ; TUNIT-SAME: (i32 returned [[X:%.*]]) #[[ATTR0:[0-9]+]] {
 ; TUNIT-NEXT:  entry:
-; TUNIT-NEXT:    [[X_ADDR:%.*]] = alloca i32, align 4
-; TUNIT-NEXT:    store i32 [[X]], ptr [[X_ADDR]], align 4
+; TUNIT-NEXT:    [[X_ADDR1:%.*]] = alloca i8, i32 4, align 4
----------------
nikic wrote:

Not sure where these changes are coming from, but this is non-canonical. You need to generate `alloca [4 x i8]` instead.

https://github.com/llvm/llvm-project/pull/72029


More information about the llvm-commits mailing list