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

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

In D82998#2141649 <https://reviews.llvm.org/D82998#2141649>, @dmgreen wrote:

> 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.


yeah that should be fine, the main thing I was worried about is compile-time here, but it sounds like we are good there.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82998