[PATCH] D45842: [Reassociate] swap binop operands to increase factoring potential

[Bjorn Pettersson via Phabricator via llvm-commits]

bjope added a comment.

As far as I can see this is another case of reusing existing Instructions/Values, while changing the actual value that the Instruction produce, right?

Take a look at the debug-info fixes I made here for a similar problem in RewriteExprTree: https://reviews.llvm.org/D45975
I suspect that you may need to discard debug-info in a similar way as in https://reviews.llvm.org/D45975, somewhere inside your new function swapOperandsToMatchBinops.


https://reviews.llvm.org/D45842