[PATCH] D113442: [InstCombine] Enable fold select into operand for FAdd, FMul, FSub and FDiv.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 1 08:57:40 PST 2022


RKSimon added a comment.

In D113442#3287665 <https://reviews.llvm.org/D113442#3287665>, @xbolva00 wrote:

> In D113442#3206100 <https://reviews.llvm.org/D113442#3206100>, @goncharov wrote:
>
>> Just FYI: I found that this change might affect of Ptrdist-ks benchmark from MultiSource suite on AMD Rome cores up to 30%. Don't have much details but leaving it here in case anyone else is affected.
>
> This is huge regression. Does @spatel ´s patch fix it?

D118644 <https://reviews.llvm.org/D118644> only handles vectors and AVX512 targets so this regression wouldn't be fixed


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113442



More information about the llvm-commits mailing list