[PATCH] D18419: Handle section vs global name conflict.

Peter Collingbourne via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 23 18:54:42 PDT 2016


pcc added inline comments.

================
Comment at: include/llvm/MC/MCContext.h:101
@@ -100,2 +100,3 @@
     StringMap<bool, BumpPtrAllocator &> UsedNames;
+    StringMap<bool, BumpPtrAllocator &> UsedSectionNames;
 
----------------
I don't think you need to add this field. You could just use `UsedNames` for section names and use the bool value to keep track of whether the symbol name has been used for a non-section. The bool appears to be unused at the moment [1].

[1] http://llvm-cs.pcc.me.uk/include/llvm/MC/MCSymbol.h/rgetNameEntryPtr http://llvm-cs.pcc.me.uk/include/llvm/MC/MCSymbol.h/rgetNameEntryPtr-1


Repository:
  rL LLVM

http://reviews.llvm.org/D18419





More information about the llvm-commits mailing list