[PATCH] D14707: add fast-math-flags to 'call' instructions (PR21290)

[Akira Hatanaka via llvm-commits]

ahatanak added inline comments.

================
Comment at: lib/AsmParser/LLParser.cpp:5647
@@ +5646,3 @@
+///       OptionalAttrs Type Value ParameterList OptionalAttrs
+///   ::= 'notail' 'call' OptionalFastMathFlags OptionalCallingConv
+///       OptionalAttrs Type Value ParameterList OptionalAttrs
----------------
Is it possible to commit the 'notail' change in a separate patch (and thank you for catching this)?

================
Comment at: lib/AsmParser/LLParser.cpp:5678
@@ -5670,1 +5677,3 @@
 
+  if (FMF.any() && !RetType->isFPOrFPVectorTy())
+    return Error(CallLoc, "fast-math-flags specified for invalid call");
----------------
Are there cases where you want to optimize calls to functions that take floating point arguments but don't have floating point return types?


http://reviews.llvm.org/D14707