[PATCH] D109441: [FuncSpec] Don't specialise call sites that have the MinSize attribute set

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 9 03:52:30 PDT 2021


SjoerdMeijer updated this revision to Diff 371543.
SjoerdMeijer added a comment.

I have left a comment in rewriteCallSites, and added an extra `test function-specialization-minsize3.ll` that contains 2 calls with only one of them annotated with MinSize to make it even clearer that we only rewrite the callsites that don't have minsize set.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109441/new/

https://reviews.llvm.org/D109441

Files:
  llvm/lib/Transforms/IPO/FunctionSpecialization.cpp
  llvm/test/Transforms/FunctionSpecialization/function-specialization-minsize2.ll
  llvm/test/Transforms/FunctionSpecialization/function-specialization-minsize3.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D109441.371543.patch
Type: text/x-patch
Size: 3684 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210909/5b474318/attachment.bin>


More information about the llvm-commits mailing list