[PATCH] D31085: [InlineCost] Increase the cost of Switch
Hans Wennborg via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 11 14:18:53 PDT 2017
hans added a comment.
I was hoping we wouldn't need the refactoring to SwitchCaseCluster.cpp, and that InlineCost.cpp could just work with TLI to check whether jump tables are allowed, density requirements etc.
My concern is that separating the case cluster code from SelectionDAGBuilder might be more trouble than it's worth.
https://reviews.llvm.org/D31085
More information about the llvm-commits
mailing list