[PATCH] D31080: [DAG] Extract switch lowering as a spearate object NFC

[Hans Wennborg via Phabricator via llvm-commits]

hans added a comment.

Hi Jun,

I think extracing the logic for clustering cases into jump tables, bit tests, etc. into a separate class might be a good idea, but I don't think we should extract the actual lowering parts. If the purpose of this is to expose a hook to figure out how a switch table will be lowered, the actual lowering doesn't need to happen here. The class that deals with clustering cases should not need to know about SDAGBuilder.

I'm concerned that this hook might turn out to be very expensive though.


https://reviews.llvm.org/D31080