[llvm] [NFC] Move RemoveRedundantDbgInstrs outside of inner loop in JumpThreading (PR #123008)

[Stephen Tozer via llvm-commits]

================
@@ -92,10 +92,6 @@ exit:                                             ; preds = %bb.f4, %bb.f3, %bb.
   ret void, !dbg !29
 }
 
-; Test for the cloning of dbg.values on elided instructions -- down one path
-; being threaded, the `and` in the function below is optimised away, but its
-; debug-info should still be preserved.
-; Similarly, the call to f1 gets cloned, its dbg.value should be cloned too.
----------------
SLTozer wrote:

I don't think this comment should be removed - the reference to "cloning" here isn't about the duplicated adjacent #dbg_values, but to the #dbg_values appearing in the new block created by jump threading.

https://github.com/llvm/llvm-project/pull/123008