The check of the possible dead node can be performed via SDValue::use_empty() instead of SDNode::use_empty().
The API is actually different: SDNode::use_empty checks uses while SDValue::use_empty only checks uses of the return value. The former is more efficient. I think it is possible that SDValue::use_empty can be used instead but I am not sure this entails a change.
You can click "Add Action..." and then abandon the differential revision.