[PATCH] D13138: [ELF2] - add support of --noinhibit-exec flag

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 24 09:42:00 PDT 2015


ruiu added inline comments.

================
Comment at: ELF/Writer.cpp:307-309
@@ +306,5 @@
+
+  Twine Error =
+      Twine("undefined symbol: ") +
+      (SymFile ? (Sym.getName() + " in " + SymFile->getName()) : Sym.getName());
+  if (Config->NoInhibitExec)
----------------
"A Twine is not intended for use directly and should not be stored, its implementation relies on the ability to store pointers to temporary stack objects which may be deallocated at the end of a statement" (http://llvm.org/docs/doxygen/html/classllvm_1_1Twine.html#details)

So make this an std::string rather than a Tiwne.


http://reviews.llvm.org/D13138





More information about the llvm-commits mailing list