[PATCH] D25697: Make the LTO comdat api more symbol table friendly

[Davide Italiano via llvm-commits]

davide added a comment.

Thank you for taking care of my comments. I'm happy with it. Mehdi, what do you think?



================
Comment at: include/llvm/LTO/LTO.h:185-188
+    // Returns the index of the comdat this symbol is in or -1 if the symbol
+    // is not in a comdat.
+    // FIXME: We have to return Expected<int> because of the way aliases are
+    // implemented and the lack of a IR symbol table.
----------------
Uh, this is not ideal. Hopefully we can get rid of it once we migrate to a bitcode symtab.


https://reviews.llvm.org/D25697