[PATCH] D69066: Add support for intrinsic llvm.ppc.eieio

[Lei Huang via Phabricator via llvm-commits]

lei added inline comments.
Herald added a subscriber: wuzish.


================
Comment at: llvm/lib/Target/PowerPC/PPCInstrInfo.td:2207
+                           "eieio", IIC_LdStLoad, []>;
+
 def : Pat<(int_ppc_sync),   (SYNC 0)>, Requires<[HasSYNC]>;
----------------
This used to be under `PowerPC Instructions used for assembler/disassembler only`.  Is this no longer true?   You seem to have moved this into `PPC32 Store Instructions` section.  What does this intrinsic do?

We generally don't move unrelated code around.   Maybe keep this where it was and add your new def under `PowerPC Assembler Instruction Aliases`?


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

https://reviews.llvm.org/D69066