[PATCH] D71190: [ARM][MVE] Add complex vector intrinsics

Dave Green via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 9 05:05:00 PST 2019


dmgreen added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMISelDAGToDAG.cpp:2537
+/// Convert an SDValue to a boolean value. SDVal must be a compile-time constant
+static bool SDValueToConstBool(SDValue SDVal) {
+  ConstantSDNode *SDValConstant = dyn_cast<ConstantSDNode>(SDVal);
----------------
How come there is so much C++ code here, as opposed to using tablegen like all the other intrinsics?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71190





More information about the cfe-commits mailing list