[PATCH] D112699: [SimplifyCFG] Simplify conditions of branches

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 8 20:18:55 PST 2021


mkazantsev added a comment.

In D112699#3115197 <https://reviews.llvm.org/D112699#3115197>, @chill wrote:

> Look like some of the tests no longer test what they were supposed to test. Some of these could be rewritten in MIR, I suppose.
> And/or would it make sense to add test runs with `Options.SimplifyCondBranch` set to `false` (somehow), so old tests are not affected?

And which of them do you think now went completely astray? Skimming through this, I noticed just split-loop.ll which doesn't make sense anymore. What else do you think should be preserved?


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

https://reviews.llvm.org/D112699



More information about the llvm-commits mailing list