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

[Duncan P. N. Exon Smith via Phabricator via llvm-commits]

dexonsmith 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


Looking at existing testcases in `test/LTO`, it seems like the existing practice is to check in a `.ll` file without a summary, and generate the summary using `opt`.  Something like:

  ; RUN: opt -module-summary -o %t1.bc %s

That's probably better than my original suggestion anyway.


Repository:
  rL LLVM

https://reviews.llvm.org/D51497