[PATCH] D62266: [DAGCombine][X86][AArch64][ARM] (C - x) + y -> (y - x) + C fold

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 30 01:01:08 PDT 2019


lebedev.ri added a comment.

In D62266#1522025 <https://reviews.llvm.org/D62266#1522025>, @dmgreen wrote:

> > *This* code? As in *this* patch, correct?
>
> Hello. I meant this .ll file seems to loop forever. With this patch or one of the three others (D62257 <https://reviews.llvm.org/D62257>, D62392 <https://reviews.llvm.org/D62392>, D62450 <https://reviews.llvm.org/D62450>).  I only tested them together, so it may be the same as the problem from the test-suite.


Confirmed, thank you for the reproducer.
This is another hang from what i have seen with D62257 <https://reviews.llvm.org/D62257>.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62266/new/

https://reviews.llvm.org/D62266





More information about the llvm-commits mailing list