[PATCH] D31013: CodeGen: BlockPlacement: Reduce TriangleChainCount to 2

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 15 20:13:49 PDT 2017


hfinkel added a comment.

In https://reviews.llvm.org/D31013#702507, @echristo wrote:

> In https://reviews.llvm.org/D31013#702483, @hfinkel wrote:
>
> > Could you make a test case? Could be an MIR test case if an IR case would be too fragile.
>
>
> Not necessarily disagreeing, but MIR and "not too fragile"? :)


Yea, okay :) -- In any case, it looks like this was committed in r297925 and there were indeed some test-case changes.


Repository:
  rL LLVM

https://reviews.llvm.org/D31013





More information about the llvm-commits mailing list