[PATCH] D131955: [InstSimplify] Remove and(x.c) if all the known bits are handled just by x or c

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 16 07:13:15 PDT 2022


RKSimon abandoned this revision.
RKSimon added a comment.

In D131955#3726046 <https://reviews.llvm.org/D131955#3726046>, @spatel wrote:

> I tried a compile-time-tracker experiment with this patch applied, and it shows a slight, but noticeable compile-time hit, so I'd say it's not worth doing for a theoretical win:
> https://llvm-compile-time-tracker.com/?config=NewPM-O3&stat=instructions&remote=rotateright

Agreed - I'll abandon this for now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131955



More information about the llvm-commits mailing list