[PATCH] D62625: LFTR for multiple exit loops

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added a comment.

Hello,
I am going to be a bit unhelpful here. We see some regressions caused by this change, but I didn't have a chance to look into it today and so I don't have a reproducer. And here's my unhelpful question: are you aware or have you seen regressions, or are you still working on this? I am not really familiar with this pass, but from reading the description, it should do good things. Taking a wild guess, I see changes in the tests and they change comparisons from ult to ne, could it for example be that this confuses other passes/analysis?
Anyway, I will try to look into this tomorrow.
Cheers.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62625