[PATCH] D82254: [RISCV] Simplify calling use_empty()

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 20 10:34:26 PDT 2020


MaskRay added a comment.

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.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82254/new/

https://reviews.llvm.org/D82254





More information about the llvm-commits mailing list