[PATCH] D20116: Add nosideeffects function attribute

Tom Stellard via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 13 21:39:01 PDT 2016


tstellarAMD updated this revision to Diff 63914.
tstellarAMD added a comment.

Rename the attribute to speculatable.  This simplifies the patch a lot since 
we are no longer trying to solve the problem  where specifying one of
the memory properties for intrinsics implies that it has no side effects
(this should probably still be addressed in another patch).

I wasn't sure exactly how to define the speculatable attribute
in the language ref, so I copied the definition from the
isSafeToSpeculativelyExecute() function.


http://reviews.llvm.org/D20116

Files:
  docs/LangRef.rst
  include/llvm/Bitcode/LLVMBitCodes.h
  include/llvm/IR/Attributes.td
  include/llvm/IR/Function.h
  include/llvm/IR/Intrinsics.td
  lib/AsmParser/LLLexer.cpp
  lib/AsmParser/LLParser.cpp
  lib/AsmParser/LLToken.h
  lib/Bitcode/Reader/BitcodeReader.cpp
  lib/Bitcode/Writer/BitcodeWriter.cpp
  lib/IR/Attributes.cpp
  lib/IR/Verifier.cpp
  test/Bitcode/attributes.ll
  test/Bitcode/compatibility.ll
  utils/TableGen/CodeGenIntrinsics.h
  utils/TableGen/CodeGenTarget.cpp
  utils/TableGen/IntrinsicEmitter.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D20116.63914.patch
Type: text/x-patch
Size: 10924 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160714/4bd38252/attachment.bin>


More information about the llvm-commits mailing list