[lld] [LLD][COFF] Swap the meaning of symtab and hybridSymtab in hybrid images (PR #135093)

Martin Storsjö via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 11 04:11:48 PDT 2025


================
@@ -32,7 +32,7 @@ class COFFLinkerContext : public CommonLinkerContext {
   SymbolTable symtab;
   COFFOptTable optTable;
 
-  // A hybrid ARM64EC symbol table on ARM64X target.
+  // A native ARM64 symbol table on ARM64X target.
   std::optional<SymbolTable> hybridSymtab;
----------------
mstorsjo wrote:

I wonder if a different name than `hybridSymtab` would be more accurate after this step? Then again, I guess hybrid means the combination of arm64ec and arm64, so in that sense I guess "hybrid" still is relevant even if we mean the other one. (I.e. "the other symtab, for the cases when we have two".)

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


More information about the llvm-commits mailing list