[PATCH] D51777: [IndVars] Set Changed if sinkUnusedInvariants changes IR. PR38863

[Serguei Katkov via Phabricator via llvm-commits]

skatkov added a comment.

I see that IndVarsSimplify uses class field Changed. You propose to use Changed as return value. I just want to see the same approach through the whole IndVars...

Do you plan to modify other pieces to return Changed or may be it makes sense to re-use the class field approach here as well?
What do you think?


https://reviews.llvm.org/D51777