[PATCH] D34728: [ThinkLTO] Invoke build(Thin)?LTOPreLinkDefaultPipeline.
Teresa Johnson via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jun 27 18:36:40 PDT 2017
tejohnson added inline comments.
================
Comment at: llvm/test/Other/new-pm-thinlto-defaults.ll:157
; CHECK-PRELINK-O-NEXT: Running pass: GlobalOptPass
-; CHECK-PRELINK-O-NEXT: Running pass: NameAnonGlobalPass
; CHECK-POSTLINK-O-NEXT: Running pass: PassManager<{{.*}}Module{{.*}}>
----------------
Can this be changed to check for the pass being added in its new location, since it should still be invoked somewhere for ThinLTO? If this change means it is no longer added under options to set up the thinlto pipeline via opt, I'd prefer that we go back to adding this to the pipeline in buildThinLTOPreLinkDefaultPipeline in the non-O0 case.
https://reviews.llvm.org/D34728
More information about the cfe-commits
mailing list