[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 17:36:57 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:
> 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
(and r290740)


https://reviews.llvm.org/D28169





More information about the llvm-commits mailing list