[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