From: Craig Topper Date: Sat, 5 Nov 2016 04:00:31 +0000 (+0000) Subject: [X86] Remove broken support for autoupgrading llvm.x86.fma4.* intrinsics to llvm... X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=a2f12e0a7511ef0c1979a5b9258a1a0101ba4eb0;p=platform%2Fupstream%2Fllvm.git [X86] Remove broken support for autoupgrading llvm.x86.fma4.* intrinsics to llvm.x86.fma.*. It currently fires an assert if you even try. Looking back, I don't think it ever worked because it only changed the name of the function object, but not the intrinsic ID stored in it. Given that, I think it can be removed since no one has noticed or complained in the past 4 years. llvm-svn: 286031 --- diff --git a/llvm/lib/IR/AutoUpgrade.cpp b/llvm/lib/IR/AutoUpgrade.cpp index a5447cb4..e6b80c5 100644 --- a/llvm/lib/IR/AutoUpgrade.cpp +++ b/llvm/lib/IR/AutoUpgrade.cpp @@ -403,12 +403,6 @@ static bool UpgradeIntrinsicFunction1(Function *F, Function *&NewFn) { NewFn = Intrinsic::getDeclaration(F->getParent(), ShiftID); return true; } - // Fix the FMA4 intrinsics to remove the 4 - if (IsX86 && Name.startswith("fma4.")) { - rename(F); - NewFn = F; - return true; - } // Upgrade any XOP PERMIL2 index operand still using a float/double vector. if (IsX86 && Name.startswith("xop.vpermil2")) { auto Params = F->getFunctionType()->params();