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

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 6 05:06:20 PDT 2021


spatel added a comment.

In D96945#2669224 <https://reviews.llvm.org/D96945#2669224>, @rupprecht wrote:

> @spatel c590a9880d7a660a1c911fce07f3d01ea18be2df <https://reviews.llvm.org/rGc590a9880d7a660a1c911fce07f3d01ea18be2df> fixes the unreduced test case. Thanks!

Great! We also have essentially the same bug inside of InstSimplify, and that should be fixed with:
e2a0f512eaca <https://reviews.llvm.org/rGe2a0f512eacad0699be9660f668726d7deb2cd75>

I'm not sure exactly where we stand with respect to exposure/timing for the 12.0 release, but I've marked the patches as blockers for now. 
That is tracked at:
https://llvm.org/PR49832


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