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

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

Thanks.

I had started collecting some statistics. It appears this patch lowers the number of calls to alias(..) by 0.8%, which matches up with what @nikic was seeing. The percentage of calls that end up as No/Must/Partial alias goes up from 65.7% to 66%. Which probably doesn't tell you a lot on it's own considering all the things that could be going on, but it's in the right direction.

Like I said, my benchmarks show this to be an improvement, and it should give better aliasing info. If it does cause problems for anyone I am of course happy to take look.


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

https://reviews.llvm.org/D82998