[PATCH] D41860: [CallSiteSplitting] Support splitting of blocks with instrs before call.

[Jun Bum Lim via Phabricator via llvm-commits]

junbuml added a subscriber: hfinkel.
junbuml added a comment.

> I've updated the code to use getInstructionCost with TCK_CodeSize. Per default it uses getUserCost, which seems OK, although in some cases, 
>  the cost it returns might be too high for our case (e.g. sdiv has a high cost although it might be lowered to a single instruction)

If getUserCost is supposed to represent only the code size (I believe so), then we may need to introduce a new target hook to handle sdiv properly in getOperationCost(). @hfinkel might have better idea about it.


https://reviews.llvm.org/D41860