[PATCH] D129980: [RISCV] Optimize (seteq (i64 (and X, 0xffffffff)), C1)

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 18 08:59:49 PDT 2022


reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.

LGTM

Correct me if I'm wrong, but this should improve the idiomatic icmp eq i32 patterns for ELEN=64 right?  If so, have you thought about analogous cases for 16 and 8 bits w/Zb*?  On the surface, using a sext.h/b over a zext.h/b wouldn't seem to matter, but the constant simplifications are potentially interesting.  Should we be canonicalizing towards sext anyways?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129980



More information about the llvm-commits mailing list