[clang] [llvm] [MTE] Apply alignment / size in AsmPrinter rather than IR (PR #111918)

via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 21 15:41:19 PDT 2024


================
@@ -764,11 +764,18 @@ void AsmPrinter::emitGlobalVariable(const GlobalVariable *GV) {
 
   const DataLayout &DL = GV->getDataLayout();
   uint64_t Size = DL.getTypeAllocSize(GV->getValueType());
+  if (GV->isTagged())
+    Size = alignTo(Size, 16);
----------------
pcc wrote:

Wouldn't we need to add padding to the initializer as well to replace the code you're removing in AArch64GlobalsTagging.cpp? I think in the usual case (non-bss) all this would do is change the value of the st_size field in the symbol.

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


More information about the cfe-commits mailing list