[PATCH] D69562: Mapping of FP operations to constrained intrinsics

Kevin P. Neal via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 6 10:09:07 PST 2019


kpn added inline comments.


================
Comment at: llvm/lib/IR/FloatingPoint.cpp:81
 
+Intrinsic::ID getConstrainedIntrinsic(const Instruction &Instr) {
+  Intrinsic::ID IID = Intrinsic::not_intrinsic;
----------------
For something that's not part of a class, shouldn't it have a more specific name? Maybe getConstrainedIntrinsicID()?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69562





More information about the llvm-commits mailing list