[PATCH] D82998: [BasicAA] Enable -basic-aa-recphi by default
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 9 07:00:37 PDT 2020
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
More information about the llvm-commits
mailing list