[llvm] [SystemZ][z/OS] Support emitting common symbols in HLASM (PR #215278)

Maryam Moghadas via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 10 11:47:59 PDT 2026


================
@@ -73,33 +73,38 @@ bool MCGOFFStreamer::emitSymbolAttribute(MCSymbol *Sym,
   return static_cast<MCSymbolGOFF *>(Sym)->setSymbolAttribute(Attribute);
 }
 
-void MCGOFFStreamer::emitCommonSymbol(MCSymbol *S, uint64_t Size,
-                                      Align ByteAlignment) {
-  auto *Symbol = static_cast<MCSymbolGOFF *>(S);
-
-  MCSectionGOFF *SD = getContext().getGOFFSection(
+void MCGOFFStreamer::emitCommonSymbolImpl(MCStreamer *Streamer,
+                                          MCSymbolGOFF *Symbol, uint64_t Size,
----------------
maryammo wrote:

Nit: could `emitCommonSymbolImpl` take `MCSymbol *` instead of `MCSymbolGOFF *`, and do the static_cast internally? That would remove the duplicate cast at both call sites.

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


More information about the llvm-commits mailing list