[PATCH] D70096: [strictfp] Replace dangling strictfp attrs with nobuiltin

Andy Kaylor via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 11 11:45:15 PST 2019


andrew.w.kaylor created this revision.
andrew.w.kaylor added reviewers: kpn, cameron.mcinally, uweigand, craig.topper, pcc.
Herald added subscribers: llvm-commits, hiraditya.
Herald added a project: LLVM.

In preparation for a patch that will enforce new rules for the usage of the strictfp attribute, this patch introduces auto-upgrade behavior that
will replace the strictfp attribute on callsites with nobuiltin if the enclosing function declaration doesn't also have the strictfp attribute.

      

This auto-upgrade isn't being performed on .ll files because that would prevent us from writing a test for the forthcoming verifier behavior.


https://reviews.llvm.org/D70096

Files:
  llvm/include/llvm/IR/AutoUpgrade.h
  llvm/lib/Bitcode/Reader/BitcodeReader.cpp
  llvm/lib/IR/AutoUpgrade.cpp
  llvm/test/Bitcode/compatibility-5.0.ll
  llvm/test/Bitcode/compatibility-6.0.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D70096.228744.patch
Type: text/x-patch
Size: 5316 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191111/60a239ed/attachment.bin>


More information about the llvm-commits mailing list