[PATCH] D82998: [BasicAA] Enable -basic-aa-recphi by default
Hal Finkel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 9 04:31:24 PDT 2020
hfinkel accepted this revision.
hfinkel added a comment.
This revision is now accepted and ready to land.
Given that compile time looks okay (and it's clearly doing something, as its causing more simplification (or, at least, fewer instructions)), as does correctness, this LGTM. If we notice any significant performance problems, we can revert (but given that we're relatively close to branching, we should commit this sooner rather than later to give the maximum opportunity to spot such things).
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82998/new/
https://reviews.llvm.org/D82998
More information about the llvm-commits
mailing list