In D102463#2777867, @aeubanks wrote:

looks good, thanks!
can you land this yourself or do you need help doing that?

Addressed the comments. Changed functions to static and made testcase changes

Made the suggested change. Added a simple testcase which uses -print-after for LoopInterchange pass (a LoopNest pass). This would crash without this patch.

Passing the outermost loop from LoopPassManager to the call backs in case of LoopNest pass.

In D102463#2760201, @aeubanks wrote:

I thought we were going to make changes to always pass the Loop (as mentioned in the description), and not do this patch

This revision is updated with the expected change in the testcase test/Transforms/InstCombine/fdiv-to-fmul-opt.ll. Other comments are addressed too.