[PATCH] D23269: AArch64: Move remaining target specific BranchRelaxation bits to TII

Tim Northover via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 16 10:20:30 PDT 2016


t.p.northover added a comment.

Sorry I dropped this last week. It doesn't seem unreasonable to add KnownOffset arguments to InsertBranch's interface, and the return value really doesn't seem like a blocker to using InsertBranch.

In fact, I've just noticed that ReverseBranchCondition already exists. So I think all that'd be needed is the getUnconditionalCondition.

Would you like me to put together an alternative patch, and we can see if it'll fit your needs too?


https://reviews.llvm.org/D23269





More information about the llvm-commits mailing list