[PATCH] D28169: [ThinLTO] Subsume all importing checks into a single flag

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 29 16:56:34 PST 2016


mehdi_amini added inline comments.


================
Comment at: test/ThinLTO/X86/autoupgrade.ll:11
-; CHECK-NOT: 'llvm.invariant.start'
-; CHECK: record string = 'llvm.invariant.start.p0i8'
 ; CHECK-NOT: 'llvm.invariant.start'
----------------
mehdi_amini wrote:
> This was an annoying an subtle bug, I'm concerned about losing coverage for this.
> If there is no other way, we should regenerate the bitcode file with the new summary.
> I'm wondering now if this test can't be triggered with llvm-link instead of llvm-lto. I'll try and report.
r290736


https://reviews.llvm.org/D28169





More information about the llvm-commits mailing list