[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