diff options
| author | Craig Topper <craig.topper@sifive.com> | 2023-08-17 15:13:35 -0700 |
|---|---|---|
| committer | Craig Topper <craig.topper@sifive.com> | 2023-08-17 15:13:36 -0700 |
| commit | 846fbb06b8bf154a6c39d6b95a09b42ecd871811 (patch) | |
| tree | d35ec4f5446a48f08550554e75f188f069c7d42e /llvm/tools/llvm-objdump/llvm-objdump.cpp | |
| parent | ebb2e5ebb2a669824499fc074d948cd28a831151 (diff) | |
| download | llvm-846fbb06b8bf154a6c39d6b95a09b42ecd871811.zip llvm-846fbb06b8bf154a6c39d6b95a09b42ecd871811.tar.gz llvm-846fbb06b8bf154a6c39d6b95a09b42ecd871811.tar.bz2 | |
[DAGCombiner][RISCV] Return SDValue(N, 0) instead of SDValue() after 2 calls to CombineTo in visitSTORE.
RISC-V found a case where the CombineTo caused N to be CSEd with
an existing node and then deleted. The top level DAGCombiner loop
was surprised to find a node was deleted, but SDValue() was returned
from the visit function.
We need to return SDValue(N, 0) to tell the top level loop that
a change was made, but the worklist updates were already handled.
Fixes #64772.
Reviewed By: arsenm
Differential Revision: https://reviews.llvm.org/D158208
Diffstat (limited to 'llvm/tools/llvm-objdump/llvm-objdump.cpp')
0 files changed, 0 insertions, 0 deletions