Skip to content

Commit

Permalink
[RISCV] Correct the output chain in lowerFixedLengthVectorMaskedLoadT…
Browse files Browse the repository at this point in the history
…oRVV

We returned the input chain instead of the output chain from the
new load. This bypasses the load in the chain. I haven't found a
good way to test this yet. IR order prevents my initial attempts
at causing reordering.
  • Loading branch information
topperc committed Mar 18, 2021
1 parent d10f173 commit c9861f7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion llvm/lib/Target/RISCV/RISCVISelLowering.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3217,7 +3217,7 @@ SDValue RISCVTargetLowering::lowerFixedLengthVectorMaskedLoadToRVV(
Load->getMemoryVT(), Load->getMemOperand());

SDValue Result = convertFromScalableVector(VT, NewLoad, DAG, Subtarget);
return DAG.getMergeValues({Result, Load->getChain()}, DL);
return DAG.getMergeValues({Result, NewLoad.getValue(1)}, DL);
}

SDValue RISCVTargetLowering::lowerFixedLengthVectorMaskedStoreToRVV(
Expand Down

0 comments on commit c9861f7

Please sign in to comment.