[PATCH] D130075: [InstCombine] Try not to demand low order bits for Add

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 22 08:44:15 PDT 2022


foad added a comment.

> But I'm not seeing test coverage for the change here - the instcombine test is already folded without this patch:
> https://alive2.llvm.org/ce/z/RtXYQq

Yeah, sorry, that was confusing because the test was already being folded by the very transform that I'm working towards removing in D130080 <https://reviews.llvm.org/D130080>. I've added a couple of new tests that should better demonstrate the effect of the current patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130075



More information about the llvm-commits mailing list