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

Leonard Chan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 9 13:10:57 PST 2021


leonardchan added a comment.

In D96945#2614982 <https://reviews.llvm.org/D96945#2614982>, @aqjune wrote:

> Yep, I reverted this patch. Could you post the commandline options that is used to build LLVM?
> There was a similar failure reported (D95026 <https://reviews.llvm.org/D95026>) which is due to an existing bug in InstructionSimplify; maybe I can check whether its prospective fix resolves this issue as well.

Thanks for the followup. Can do. Let me see if I can minimize the reproducer/reduce the number of cmake flags needed for ease first since the two-stage build has dependencies on libxml2 and zlib.

While I'm doing that, if you already have a fix, what I can do is patch it locally and get back to you on if it also fixes the issues we see.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96945



More information about the llvm-commits mailing list