[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