[PATCH] D81236: Correctly report modified status for LoopExtractor

[serge via Phabricator via llvm-commits]

serge-sans-paille added a comment.



> Agreed. I tried to find out where LoopInfoWrapperPass would do this but I didn't spot it right away. @serge-sans-paille do you know where the modification happens or do you happen to have the reproducer?

I investigated, and BreakCriticalEdges is a dependency of LoopExtractor, and it's the pass that triggers the change. I supsect LoopSimplify, another analysis dep, has the same impact.
I don't understand where passes marked as dependency register their change status...


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

https://reviews.llvm.org/D81236