[PATCH] D155556: [GlobalISel] GIntrinsic subclass to represent intrinsics in Generic Machine IR

Sameer Sahasrabuddhe via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 18 22:59:50 PDT 2023


sameerds marked 2 inline comments as done.
sameerds added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/MachineInstr.h:1944
+  /// Returns Intrinsic::ID if this is an intrinsic, else returns not_intrinsic.
   unsigned getIntrinsicID() const {
+    if (!isIntrinsic())
----------------
arsenm wrote:
> sameerds wrote:
> > @arsenm, do you see any value in introducing `GIntrinsic` as a subclass of `GenericMachineInstr` and moving `getIntrinsicID()` to it? I tried half of that in D154766, but I don't see any real advantage. I am thinking I'll roll it back in D154766.
> We should probably do it and make more use of GenericMachineInstr
Added the new `GIntrinsic` class and updated the review description.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155556



More information about the llvm-commits mailing list