[PATCH] D86347: [SimplifyCFG] Two entry phi select costs

[Sam Parker via Phabricator via llvm-commits]

samparker updated this revision to Diff 290233.
samparker added a comment.
Herald added a subscriber: wenlei.

Rebased after moving the select cost and validation into a standalone function. I've kept the threshold at 4 as a default, because this causes far fewer codegen changes (X86, AArch64, Arm) than trying to adjust it to accommodate the select by bumping it up to 5.


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

https://reviews.llvm.org/D86347

Files:
  llvm/lib/Transforms/Utils/SimplifyCFG.cpp
  llvm/test/Transforms/PGOProfile/chr.ll
  llvm/test/Transforms/SimplifyCFG/ARM/phi-eliminate.ll
  llvm/test/Transforms/SimplifyCFG/ARM/speculate-math.ll
  llvm/test/Transforms/SimplifyCFG/PhiEliminate3.ll
  llvm/test/Transforms/SimplifyCFG/speculate-math.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D86347.290233.patch
Type: text/x-patch
Size: 52889 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200907/6bccf60e/attachment.bin>