[PATCH] D51497: [WIP][ORC][ThinLTO] Early ThinLTO-JIT prototype and basic tests for next-gen ORC APIs

Fri Aug 31 06:56:41 PDT 2018

tejohnson added a comment.

In https://reviews.llvm.org/D51497#1220621, @sgraenitz wrote:

> In https://reviews.llvm.org/D51497#1219835, @dexonsmith wrote:
>
> > You should check in LLVM textual IR files (`.ll`) instead of C++ source files or object files.  Using `clang -S -emit-llvm` (or `llvm-dis some-thinlto-bitcode-file.o`) should get you started.
>
>
> Yes that would be great, but wouldn't it require IR representation for summaries? I didn't double-check myself, but there's a note in the language ref, "that temporarily the summary entries are skipped when parsing the assembly". Not sure if I understand that correctly, but I was under the impression that we can't have summaries in IR files yet.
>  https://llvm.org/docs/LangRef.html#thinlto-summary

This is stale - Sorry, I totally forgot to remove when I implemented the parsing side, which went in a couple months ago (r335602). For an example with a summary, see test/Assembler/thinlto-summary.ll. You need to use llvm-as to assembly it (not opt). Eventually other tools such as llvm-lto2 should be able to read assembly with summary directly, but I haven't had a chance to do that yet.

Repository:
  rL LLVM

https://reviews.llvm.org/D51497