[PATCH] D95822: [FE][AIX] Use i8 as guard variable type in both load and store operation

Hubert Tong via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 1 17:35:51 PST 2021


hubert.reinterpretcast added inline comments.


================
Comment at: clang/test/CodeGenCXX/aix-static-init.cpp:143
 // CHECK: init.check:
-// CHECK:   %1 = call i32 @__cxa_guard_acquire(i64* @_ZGVZN5test41fEvE11staticLocal)
+// CHECK:   %1 = call i32 @__cxa_guard_acquire(i8* @_ZGVZN5test41fEvE11staticLocal)
 // CHECK:   %tobool = icmp ne i32 %1, 0
----------------
What actually does the definition of `_ZGVZN5test41fEvE11staticLocal` look like? The test seems to be missing something pertinent for the change being proposed here. I will note that the symbol is 8-bytes (and aligned accordingly) from XL.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95822/new/

https://reviews.llvm.org/D95822



More information about the cfe-commits mailing list