[PATCH] D7802: [Inliner] Use whitelist instead of blacklist when checking function attribute compatibility and make the check stricter

Akira Hatanaka ahatanak at gmail.com
Mon Aug 3 19:02:04 PDT 2015


ahatanak updated this revision to Diff 31290.
ahatanak added a comment.

Resurrecting the discussion on inliner's function attribute compatibility checking.

I rebased the patch and made the following changes:

- Changed the syntax of class "Attr" in Attributes.td. The new syntax allows specifying the c++ function that is called to check the compatibility between the attributes of the caller and callee.
- Updated cmake files.
- Add comments and renamed functions.


http://reviews.llvm.org/D7802

Files:
  include/llvm/IR/Attributes.h
  include/llvm/IR/Attributes.td
  include/llvm/IR/CMakeLists.txt
  lib/Analysis/IPA/InlineCost.cpp
  lib/IR/Attributes.cpp
  lib/IR/AttributesCompatFunc.td
  lib/IR/CMakeLists.txt
  lib/IR/Makefile
  test/Transforms/Inline/attributes.ll
  utils/TableGen/Attribute.cpp
  utils/TableGen/CMakeLists.txt
  utils/TableGen/TableGen.cpp
  utils/TableGen/TableGenBackends.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D7802.31290.patch
Type: text/x-patch
Size: 22102 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150804/d05de855/attachment.bin>


More information about the llvm-commits mailing list