[PATCH] D39304: [IR] redefine 'reassoc' fast-math-flag and add 'trans' fast-math-flag

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 31 08:47:45 PDT 2017


spatel updated this revision to Diff 120986.
spatel marked 4 inline comments as done.
spatel added a comment.

Patch updated - just changes to the LangRef text (this now attempts to subsume the similar changes in https://reviews.llvm.org/D28335) :

1. Add an extra line to the 'sqrt' semantics because that has well-defined behavior...unless the type is not IEEE.
2. Remove the 'afn' blurb for 'powi' because that's loose by definition.
3. Add the 'afn' blurb back to 'fma' because that actually could be affected.
4. Try to soften the existing libm language for all 10 'afn'-affected intrinsics while retaining some shred of accuracy. :)


https://reviews.llvm.org/D39304

Files:
  docs/LangRef.rst
  include/llvm/IR/Instruction.h
  include/llvm/IR/Operator.h
  include/llvm/Transforms/Utils/LoopUtils.h
  lib/AsmParser/LLLexer.cpp
  lib/AsmParser/LLParser.h
  lib/AsmParser/LLToken.h
  lib/Bitcode/Reader/BitcodeReader.cpp
  lib/Bitcode/Writer/BitcodeWriter.cpp
  lib/CodeGen/ExpandReductions.cpp
  lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
  lib/IR/AsmWriter.cpp
  lib/IR/Instruction.cpp
  lib/Target/AMDGPU/AMDGPUCodeGenPrepare.cpp
  lib/Target/AMDGPU/AMDGPULibCalls.cpp
  lib/Transforms/InstCombine/InstCombineAddSub.cpp
  lib/Transforms/InstCombine/InstCombineCalls.cpp
  lib/Transforms/InstCombine/InstCombineMulDivRem.cpp
  lib/Transforms/Scalar/Reassociate.cpp
  lib/Transforms/Utils/LoopUtils.cpp
  lib/Transforms/Utils/SimplifyLibCalls.cpp
  lib/Transforms/Vectorize/LoopVectorize.cpp
  lib/Transforms/Vectorize/SLPVectorizer.cpp
  test/Assembler/fast-math-flags.ll
  test/Bitcode/compatibility-3.6.ll
  test/Bitcode/compatibility-3.7.ll
  test/Bitcode/compatibility-3.8.ll
  test/Bitcode/compatibility-3.9.ll
  test/Bitcode/compatibility-4.0.ll
  test/Bitcode/compatibility-5.0.ll
  test/Bitcode/compatibility.ll
  unittests/IR/IRBuilderTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D39304.120986.patch
Type: text/x-patch
Size: 56784 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171031/2a709014/attachment.bin>


More information about the llvm-commits mailing list