[PATCH] D89938: [DAG][PowerPC] Do not overwrite flags in `getNodeIfExists` without passing `Flags` argument

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 22 13:09:05 PDT 2020


efriedma added a comment.

I think getNodeIfExists() should have the same semantics with respect to flags as getNode().

Maybe we can add a new helper "doesNodeExist()" which just returns a boolean, and doesn't mess with the flags?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89938



More information about the llvm-commits mailing list