[PATCH] D82145: [NFC][PowerPC] Legalize SREM/UREM directly on P9.

Kai Luo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 23 19:27:07 PDT 2020


lkail added a comment.

It looks not an NFC patch to me, the test predicates are changed

  CHECK: modsw -> CHECK-NOT: modsw

I'm curious if it is unchanged that `opt` is added to RUN lines while the OperationAction keeps `Custom`. If so, adding `opt` to RUN lines can be an NFC patch, and making OperationAction `Legal` can be another FC patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82145





More information about the llvm-commits mailing list