[PATCH] D96945: [InstCombine] Add simplification of two logical and/ors

[Leonard Chan via Phabricator via llvm-commits]

leonardchan added a comment.

HI,

A bisect seems to show this patch causing the test failures we see on our two-stage clang builders (https://luci-milo.appspot.com/p/fuchsia/builders/prod/clang-linux-x64/b8853364937280028160?):

  ********************
    LLVM :: Transforms/InstCombine/2011-05-28-swapmulsub.ll
    LLVM :: Transforms/InstCombine/bitcast-bigendian.ll
    LLVM :: Transforms/InstCombine/bitcast-inseltpoison.ll
    LLVM :: Transforms/InstCombine/bitcast.ll
    LLVM :: Transforms/InstCombine/icmp-custom-dl.ll
    LLVM :: Transforms/InstCombine/icmp.ll
    LLVM :: Transforms/InstCombine/shift-amount-reassociation-in-bittest-with-truncation-shl.ll
    LLVM :: Transforms/InstCombine/shift-amount-reassociation-with-truncation-shl.ll
    LLVM :: Transforms/InstCombine/sub-gep.ll
    LLVM :: Transforms/InstCombine/sub.ll
    LLVM :: Transforms/InstCombine/trunc-inseltpoison.ll
    LLVM :: Transforms/InstCombine/trunc.ll

Would you mind taking a look and sending out a fix or reverting? Thanks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96945