[PATCH] D57372: Demanded elements support for masked.load and masked.gather
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Feb 2 03:30:18 PST 2019
RKSimon added inline comments.
================
Comment at: lib/Transforms/InstCombine/InstCombineSimplifyDemanded.cpp:1455
+ DemandedPtrs.clearBit(i);
+ else if (CElt->isAllOnesValue())
+ DemandedPassThrough.clearBit(i);
----------------
We're not 100% consistent with this but for selects we try to use:
```
if (CElt->isNullValue())
DemandedPtrs.clearBit(i);
else
DemandedPassThrough.clearBit(i);
```
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57372/new/
https://reviews.llvm.org/D57372
More information about the llvm-commits
mailing list