[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



More information about the llvm-commits mailing list