[PATCH] D57718: [PPC] Adjust the computed branch offset for the possible shorter distance
Simon Dardis via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 13 10:14:08 PST 2019
sdardis added a comment.
In D57718#1393615 <https://reviews.llvm.org/D57718#1393615>, @Carrot wrote:
> In D57718#1393504 <https://reviews.llvm.org/D57718#1393504>, @jsji wrote:
> > Any testcases that can show the problem and test the fix? Thanks.
> It can only be triggered by a very large (>32KB) function body because the range of conditional branch is +/- 32KB.
> In our case, the large function body is caused by aggressive thinlto guided inlining.
> So no small test case can demonstrate the problem :(
You can use the assembly directive '.space' to create arbitrary sized basic blocks. See test/CodeGen/RISCV/branch-relaxation.ll for examples.
CHANGES SINCE LAST ACTION
More information about the llvm-commits