================
@@ -411,6 +412,13 @@ static Instruction
*convertNvvmIntrinsicToLlvm(InstCombiner &IC,
}
return nullptr;
}
+ case SPC_Fabs: {
+ if (!II->getType()->isDoubleTy())
+ return nullptr;
+ auto *Fabs = Intrinsic::getOrInsertDeclaration(
+ II->getModule(), Intrinsic::fabs, II->getType());
+ return CallInst::Create(Fabs, II->getArgOperand(0));
----------------
AlexMaclean wrote:
I've removed this transformation all together based on
https://github.com/llvm/llvm-project/pull/135644#discussion_r2045313522. I've
also updated the clang frontend to use `llvm.fabs` for this case since the
`llvm.nvvm.fabs` intrinsic has no advantage for doubles.
https://github.com/llvm/llvm-project/pull/135644
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
