[PATCH] D23067: TargetInstrInfo: add two new target hooks to analyse branch offsets

[Matt Arsenault via llvm-commits]

arsenm added a comment.

In https://reviews.llvm.org/D23067#517048, @SjoerdMeijer wrote:

> Do you mean that having function isBranchOffsetInRange is enough? I agree, but for a block placement algorithm that would possibly require invoking it a lot of times. Thus I thought exposing the ranges would be convenient.


Yes. How about you keep isBranchOffsetInRange in this patch, and add the others to your placement patch to show what you mean?


https://reviews.llvm.org/D23067