[PATCH] D33372: Infer relocation model from module flags in relocatable LTO link

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 22 13:43:08 PDT 2017


pcc accepted this revision.
pcc added a comment.
This revision is now accepted and ready to land.

LGTM with one nit



================
Comment at: lib/LTO/LTOBackend.cpp:120
 std::unique_ptr<TargetMachine>
 createTargetMachine(Config &Conf, StringRef TheTriple,
+                    const Target *TheTarget, Module &M) {
----------------
Remove TheTriple argument, it can always be computed from the module.


Repository:
  rL LLVM

https://reviews.llvm.org/D33372





More information about the llvm-commits mailing list