[all-commits] [llvm/llvm-project] 952dc2: [NFC][SimplifyCFG] Add some more tests w/ two-entr...

[Roman Lebedev via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 952dc2e561c97447f92d24f8c0f62fb4f0569e92
      https://github.com/llvm/llvm-project/commit/952dc2e561c97447f92d24f8c0f62fb4f0569e92
  Author: Roman Lebedev <lebedev.ri at gmail.com>
  Date:   2021-07-22 (Thu, 22 Jul 2021)

  Changed paths:
    A llvm/test/Transforms/SimplifyCFG/two-entry-phi-node.ll

  Log Message:
  -----------
  [NFC][SimplifyCFG] Add some more tests w/ two-entry PHI nodes and


  Commit: 7ef6f019090f3979fa345105b9ac95ac589c6cf9
      https://github.com/llvm/llvm-project/commit/7ef6f019090f3979fa345105b9ac95ac589c6cf9
  Author: Roman Lebedev <lebedev.ri at gmail.com>
  Date:   2021-07-22 (Thu, 22 Jul 2021)

  Changed paths:
    M llvm/lib/Transforms/Utils/SimplifyCFG.cpp
    M llvm/test/Transforms/SimplifyCFG/two-entry-phi-node.ll

  Log Message:
  -----------
  [SimplifyCFG] FoldTwoEntryPHINode(): bailout on inverted logical and/or (PR51149)

The logical (select) form of and/or will now be a source of problems.
We don't really account for it's inverted form, yet it exists,
and presumably we should treat it just like non-inverted form:
https://alive2.llvm.org/ce/z/BU9AXk

https://bugs.llvm.org/show_bug.cgi?id=51149 reports a reportedly-serious
perf regression that will hopefully be mitigated by this.


Compare: https://github.com/llvm/llvm-project/compare/00edae9203c9...7ef6f019090f