[PATCH] D78880: [InstCombine] use select-of-constants with set/clear bit mask patterns

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 6 08:35:29 PDT 2020


spatel added a comment.

In D78880#2022752 <https://reviews.llvm.org/D78880#2022752>, @sanwou01 wrote:

> Hi, we're seeing a small (1.0%) regression in omnetpp_r in SPEC INT 2017 on AArch64 with LTO enabled that bisects to this patch. I should be able to reduce omnetpp_r to a small IR example that shows the changed AArch64 codegen, if that's useful. A revert is probably not necessary if all we need is an additional pattern or two in the AArch64 backend.


Thanks for letting me know. Yes, an IR example would be very helpful in deciding how to proceed. 
Note: D68911 <https://reviews.llvm.org/D68911> has been sitting untouched for ~6 months...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78880





More information about the llvm-commits mailing list