[PATCH] D22892: Codegen: MachineBlockPlacement Improve probability layout.

Xinliang David Li via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 29 10:54:42 PDT 2016


you can probably split the test changes as a cleanup patch separately.

David

On Fri, Jul 29, 2016 at 10:44 AM, Kyle Butt <kyle+llvm at iteratee.net> wrote:
> iteratee updated this revision to Diff 66146.
> iteratee added a comment.
>
> Added fixes for 4 affected tests. Since they weren't testing layout, I added branch weights to get the layout that was produced before. In 3 cases, this was simply equal-weighting the branches, and in the 4th the expected layout required slight biasing.
>
>
> https://reviews.llvm.org/D22892
>
> Files:
>   lib/CodeGen/MachineBlockPlacement.cpp
>   test/CodeGen/AArch64/arm64-andCmpBrToTBZ.ll
>   test/CodeGen/AArch64/compare-branch.ll
>   test/CodeGen/AArch64/logical_shifted_reg.ll
>   test/CodeGen/SystemZ/tdc-06.ll
>   test/CodeGen/X86/block-placement.ll
>


More information about the llvm-commits mailing list