[PATCH] D82998: [BasicAA] Enable -basic-aa-recphi by default

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

It should be triggering relatively often, from looking at the tests/benchmarks I was seeing. Any code that uses pointer induction variables can trigger it. Whether that will common will be a matter of the style of the original code, and whether it actually proves move NoAlias will be variable. But it seems to come up enough to make code improvements in a fair number of cases.

>From what I can tell it should be almost free though, in terms of compile time. Just an extra check of whether a phi is a simple recursion.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82998/new/

https://reviews.llvm.org/D82998