[PATCH] D58910: [LLD][COFF] More detailed information for /failifmismatch

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added inline comments.


================
Comment at: lld/trunk/COFF/DriverUtils.cpp:701
 // if value matches previous values for the same key.
-void checkFailIfMismatch(StringRef Arg) {
+void checkFailIfMismatch(StringRef Arg, StringRef Source) {
   StringRef K, V;
----------------
In general, an error string that may be composed should be of type `const Twine &` instead of `StringRef` to avoid cost of string concatenation.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58910