[PATCH] D144596: [lld] Support separate native object file path in --thinlto-prefix-replace

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added inline comments.


================
Comment at: lld/test/ELF/lto/thinlto-index-file-object-prefix-replace.ll:41
+; RUN: not ld.lld --thinlto-prefix-replace="old/;new/;obj/" -shared old/subdir/1.o old/subdir/2.o old/3.o -o /dev/null 2>&1 | FileCheck %s --check-prefix=ERR
+; ERR: thinlto-prefix-replace
+
----------------
MaskRay wrote:
> Here and throughout, always include `error:` for an error message, and try including the input filename.
> 
I believe what @MaskRay meant was to match the "error:" from the error message that this presumably comes from (and the input filename assuming that is in the error message I guess?) - I would just go ahead and match the full error message output.


================
Comment at: llvm/tools/gold/gold-plugin.cpp:895
     Backend = createWriteIndexesThinBackend(OldPrefix, NewPrefix,
+                                            /*NativeObjectPrefix=*/"",
                                             options::thinlto_emit_imports_files,
----------------
Add a TODO to support the new option syntax used by lld in the gold plugin.


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

https://reviews.llvm.org/D144596