[PATCH] D31359: [GlobalISel]: Allow backends to custom legalize Intrinsics

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: include/llvm/CodeGen/GlobalISel/LegalizerInfo.h:195-196
 
+  virtual bool legalizeIntrinsic(MachineInstr &MI, MachineRegisterInfo &MRI,
+                                 MachineIRBuilder &MIRBuilder) const;
+
----------------
I think this need to directly return LegalizeResult. Some intrinsics may be AlreadyLegal, or Legalized


Repository:
  rL LLVM

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

https://reviews.llvm.org/D31359