[PATCH] D15668: [attrs] Split off the forced attributes utility into its own pass that can be potentially run much earlier than FuncitonAttrs proper.

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 19 12:01:14 PST 2015


chandlerc created this revision.
chandlerc added a reviewer: jmolloy.
chandlerc added a subscriber: llvm-commits.

This, for example, would allow forcing optnone or other widely impactful
attribute. It is also a bit simpler as the force attribute behavior needs no
specific iteration order.

An important question is whether it is useful to add this to the default pass
pipelines or not. I'm indifferent, happy to take suggestions. If so, it should
go at the very start so that optnone takes full effect, etc.

http://reviews.llvm.org/D15668

Files:
  include/llvm/InitializePasses.h
  include/llvm/Transforms/IPO/ForceFunctionAttrs.h
  lib/Transforms/IPO/CMakeLists.txt
  lib/Transforms/IPO/ForceFunctionAttrs.cpp
  lib/Transforms/IPO/FunctionAttrs.cpp
  lib/Transforms/IPO/IPO.cpp
  test/Transforms/ForcedFunctionAttrs/forced.ll
  test/Transforms/FunctionAttrs/forced.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D15668.43307.patch
Type: text/x-patch
Size: 12282 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151219/2f8f00eb/attachment.bin>


More information about the llvm-commits mailing list