[PATCH] D116935: [IRBuilder] Introduce folder using inst-simplify, use for Or fold.

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 10 10:46:00 PST 2022


lebedev.ri added a comment.

In D116935#3232126 <https://reviews.llvm.org/D116935#3232126>, @craig.topper wrote:

> If I remember correctly, the Or and And folds for 0 and all 1s are there to optimize bitfield codegen for the frontend. Are we losing that optimization if this patch goes in before switching clang to InstSimplifyFolder?

True, but why is that optimization there in the first place?
Presumably it shouldn't matter nowadays.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116935



More information about the cfe-commits mailing list