[PATCH] D84071: More conservatively report status from LoopIdiomRecognize

Thu Jul 23 11:06:43 PDT 2020

jroelofs added a comment.

In D84071#2169764 <https://reviews.llvm.org/D84071#2169764>, @jroelofs wrote:

> In D84071#2169440 <https://reviews.llvm.org/D84071#2169440>, @fhahn wrote:
>
> > Do you happen to still have the test case for this? If it is reasonably small, would it be possible to add to the tests?
>
>
> I have the IR, but the assert doesn't repro if I run it through the pass via opt. I'll see if I can reduce it a bit so we at least have a test where the PM reports the change.

https://reviews.llvm.org/D84450

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84071