[PATCH] D113210: [NewPM] Use the default AA pipeline by default

2021-11-05 Thread Arthur Eubanks via Phabricator via cfe-commits
aeubanks added a comment. fixed with 7f62759697762473 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113210/new/ https://reviews.llvm.org/D113210

[PATCH] D113210: [NewPM] Use the default AA pipeline by default

2021-11-05 Thread Arthur Eubanks via Phabricator via cfe-commits
aeubanks added a comment. In D113210#3110780 , @Meinersbur wrote: > This change caused the Polly build to fail: > https://lab.llvm.org/buildbot/#/builders/10/builds/7501 > > opt: >

[PATCH] D113210: [NewPM] Use the default AA pipeline by default

2021-11-05 Thread Michael Kruse via Phabricator via cfe-commits
Meinersbur added a comment. This change caused the Polly build to fail: https://lab.llvm.org/buildbot/#/builders/10/builds/7501 opt: /home/worker/buildbot-workers/polly-x86_64-gce1/rundir/llvm.src/llvm/include/llvm/IR/PassManager.h:784: typename PassT::Result&

[PATCH] D113210: [NewPM] Use the default AA pipeline by default

2021-11-04 Thread Arthur Eubanks via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG13317286f829: [NewPM] Use the default AA pipeline by default (authored by aeubanks). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D113210: [NewPM] Use the default AA pipeline by default

2021-11-04 Thread Arthur Eubanks via Phabricator via cfe-commits
aeubanks updated this revision to Diff 384842. aeubanks added a comment. update LTOBackend.cpp Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113210/new/ https://reviews.llvm.org/D113210 Files: clang/lib/CodeGen/BackendUtil.cpp

[PATCH] D113210: [NewPM] Use the default AA pipeline by default

2021-11-04 Thread Alina Sbirlea via Phabricator via cfe-commits
asbirlea added a comment. Can you also update `lib/LTO/LTOBackend.cpp` to something like: // Parse a custom AA pipeline if asked to, otherwise the default will be used. if (!Conf.AAPipeline.empty()) { AAManager AA; if (auto Err = PB.parseAAPipeline(AA, Conf.AAPipeline)) {

[PATCH] D113210: [NewPM] Use the default AA pipeline by default

2021-11-04 Thread Arthur Eubanks via Phabricator via cfe-commits
aeubanks created this revision. aeubanks added a reviewer: asbirlea. Herald added subscribers: ormris, steven_wu, hiraditya. Herald added a reviewer: ctetreau. Herald added a reviewer: ctetreau. aeubanks requested review of this revision. Herald added projects: clang, LLVM. Herald added