[llvm] [AMDGPU] Don't realign already allocated LDS. Point fix for 106412 (PR #106421)

Jon Chesterfield via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 28 10:29:50 PDT 2024


================
@@ -0,0 +1,18 @@
+; RUN: opt -S -mtriple=amdgcn-- -passes=amdgpu-lower-module-lds < %s | FileCheck %s
+
+; Can't have a second variable without absolute_symbol showing it is realigned as
+; there is a fatal error on mixing absolute and non-absolute symbols
+
+; CHECK: @lds.dont_realign_undef = internal addrspace(3) global i64 undef, align 2, !absolute_symbol !0
+; CHECK: @lds.dont_realign_poison = internal addrspace(3) global i64 poison, align 2, !absolute_symbol !0
+ at lds.dont_realign_undef = internal addrspace(3) global i64 undef, align 2, !absolute_symbol !0
----------------
JonChesterfield wrote:

yeah, race condition between review requests and updating the branch

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


More information about the llvm-commits mailing list