Skip to content

Commit

Permalink
[DAG] TransformFPLoadStorePair - early out if we're not loading a sim…
Browse files Browse the repository at this point in the history
…ple type

Its never going to transform into a legal integer type, so just bail - noticed while triaging the assertion reported in llvm#121784
  • Loading branch information
RKSimon committed Jan 7, 2025
1 parent 4e066b6 commit 1332db3
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -20455,10 +20455,8 @@ SDValue DAGCombiner::TransformFPLoadStorePair(SDNode *N) {
Value.hasOneUse()) {
LoadSDNode *LD = cast<LoadSDNode>(Value);
EVT VT = LD->getMemoryVT();
if (!VT.isFloatingPoint() ||
VT != ST->getMemoryVT() ||
LD->isNonTemporal() ||
ST->isNonTemporal() ||
if (!VT.isSimple() || !VT.isFloatingPoint() || VT != ST->getMemoryVT() ||
LD->isNonTemporal() || ST->isNonTemporal() ||
LD->getPointerInfo().getAddrSpace() != 0 ||
ST->getPointerInfo().getAddrSpace() != 0)
return SDValue();
Expand Down

0 comments on commit 1332db3

Please sign in to comment.