[PATCH] D96881: [InstCombine] Avoid Bitcast-GEP fusion for pointers directly from allocation functions

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 23 14:15:30 PDT 2021


echristo added a comment.

In D96881#2627389 <https://reviews.llvm.org/D96881#2627389>, @nikic wrote:

> LGTM per above discussion. It's not ideal, but I think it's the best we can do right now.

Hi All,

I'm seeing a rather numerous set of performance degredations and code size increases on x86 with this patch. I think that's a little unexpected, but do we have benchmarks and runs showing that this is a profitable change to happen in general?

Thanks!

-eric


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96881



More information about the llvm-commits mailing list