[PATCH] D25322: DAG: Setting Masked-Expand-Load as a variant of Masked-Load node.

[Alexey Bataev via llvm-commits]

ABataev added inline comments.


> SelectionDAG.cpp:5350
>                                      EVT MemVT, MachineMemOperand *MMO,
> -                                    ISD::LoadExtType ExtTy) {
> +                                    ISD::LoadExtType ExtTy, bool isExpand) {
>  

isExpand->IsExpanding

> SelectionDAG.cpp:5377
>                                       EVT MemVT, MachineMemOperand *MMO,
>                                       bool isTrunc, bool isCompress) {
>    assert(Chain.getValueType() == MVT::Other &&

isCompress->IsCompressing

> SelectionDAGBuilder.cpp:3859
>    SDValue Load = DAG.getMaskedLoad(VT, sdl, InChain, Ptr, Mask, Src0, VT, MMO,
> -                                   ISD::NON_EXTLOAD);
> +                                   ISD::NON_EXTLOAD, false);
>    if (AddToChain) {

`, false`->`, /*isExpanding=*/false`

> X86ISelLowering.cpp:18900
> +    return DAG.getMaskedLoad(VT, dl, Chain, Addr, VMask, PassThru, VT,
> +                             MemIntr->getMemOperand(), ISD::NON_EXTLOAD, true);
>    }

`, true`->`, /*isExpanding=*/true`

Repository:
  rL LLVM

https://reviews.llvm.org/D25322