[PATCH] D130391: revert "[CodeGen] Require a name for a block addr target"

Sergei Barannikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 22 16:10:02 PDT 2022


barannikov88 added inline comments.


================
Comment at: llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp:232
   Entry.Fn = BB->getParent();
-  MCSymbol *Sym = BB->hasAddressTaken() ? Context.createNamedTempSymbol()
-                                        : Context.createTempSymbol();
-  Entry.Symbols.push_back(Sym);
+  Entry.Symbols.push_back(Context.createTempSymbol());
   return Entry.Symbols;
----------------
The old behavior is to call `createNamedTempSymbol`
I can't judge if the new behavior is wrong or not, just noticed that it has changed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130391



More information about the llvm-commits mailing list