[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