[PATCH] D115458: [FuncSpec] Decouple Cost and Benefit analysis, to sort candidates. NFC.

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 10 03:30:29 PST 2021


SjoerdMeijer updated this revision to Diff 393431.
SjoerdMeijer retitled this revision from "[FuncSpec] Decouple Cost and Benefit analysis, to sort candidates." to "[FuncSpec] Decouple Cost and Benefit analysis, to sort candidates. NFC.".
SjoerdMeijer added a comment.

This addresses the other comments. This should now be NFC, so have added this to the Title.
I will double check the SPEC score remains the same, and will follow up with the functional change that was present of the first revision of this patch which caused the change in the test.


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

https://reviews.llvm.org/D115458

Files:
  llvm/lib/Transforms/IPO/FunctionSpecialization.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D115458.393431.patch
Type: text/x-patch
Size: 13665 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211210/cbb38866/attachment.bin>


More information about the llvm-commits mailing list