[PATCH] D151878: [GlobalIsel][X86] Update legalization of G_ADD/G_SUB

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 1 12:25:52 PDT 2023


RKSimon updated this revision to Diff 527556.
RKSimon edited the summary of this revision.
RKSimon added a comment.

Use legalIf() and split clampNumElements to simplify max num elements logic.

I'm pretty happy with this for now in terms of clarity, and it should work easily to let us remove the existing legacy legalizers and then add other legal ops on top.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151878

Files:
  llvm/lib/Target/X86/X86LegalizerInfo.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D151878.527556.patch
Type: text/x-patch
Size: 7814 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230601/6bbeacec/attachment.bin>


More information about the llvm-commits mailing list