[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