[PATCH] D102443: [PowerPC] Added multiple PowerPC builtins

Nemanja Ivanovic via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 26 08:52:00 PDT 2021


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

LGTM. Thanks for implementing this. The redundant pseudo can be removed when committing.



================
Comment at: llvm/lib/Target/PowerPC/PPCInstrInfo.td:2572
+                  [(int_ppc_eieio)]>;
+def PseudoIOSPACEEIEIO : PPCEmitTimePseudo<(outs), (ins), "#PPCIOSPACEEIEIO",
+                         [(int_ppc_iospace_eieio)]>;
----------------
These two are identical. Please only define one.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102443



More information about the cfe-commits mailing list