[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#503745, @SjoerdMeijer wrote:

> Hi Matt, thanks for reviewing. I have changed the offsets to int64_t.
>  I had a look and there indeed appears to be some overlap.
>  The motivation for my change is to minimise branch distances by reordering machine blocks, for which I need these helper functions.


I was thinking about trying to optimize the branch relaxation pass by reordering and duplicating blocks to avoid long expansions. Do you think it makes sense as a separate pass, or for BranchRelaxation to try to do both?


https://reviews.llvm.org/D23067