[PATCH] D128241: [AMDGPU] Use AddedComplexity for ret and noret atomic ops selection

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D128241#3632654 <https://reviews.llvm.org/D128241#3632654>, @abinavpp wrote:

> I don't think I understood the question. I set the AddedComplexity in such a way that no-return matches are always done first since we're only adding predicates for those. By "default complexity", are we talking about getting PatternToMatch::getPatternComplexity() in CodeGenDagPatterns to give higher complexity to patterns with HasNoUse predicates?

Yes. I thought predicates boosted the complexity, such that this would work out by default:

  // If this node has some predicate function that must match, it adds to the
  // complexity of this node.
  if (!P->getPredicateCalls().empty())
    ++Size;


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128241