[PATCH] D127392: [AggressiveInstCombine] Combine consecutive loads which are being merged to form a wider load.
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 15 00:44:20 PDT 2022
nikic added a comment.
There are a number of test failures in pre-merge checks, probably pipeline tests.
================
Comment at: llvm/lib/Transforms/AggressiveInstCombine/AggressiveInstCombine.cpp:735
auto &TTI = getAnalysis<TargetTransformInfoWrapperPass>().getTTI(F);
- return runImpl(F, AC, TTI, TLI, DT);
+ auto AA = &getAnalysis<AAResultsWrapperPass>().getAAResults();
+ return runImpl(F, AC, TTI, TLI, DT, AA);
----------------
Just like all the other analyses, this should use a reference, not a pointer (the analysis is not optional).
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D127392/new/
https://reviews.llvm.org/D127392
More information about the llvm-commits
mailing list