[all-commits] [llvm/llvm-project] f8ed60: [RDF] Define short type names: NodeAddr<XyzNode*> ...

[Krzysztof Parzyszek via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f8ed60b56d1948422dda924fcf450560591e8a19
      https://github.com/llvm/llvm-project/commit/f8ed60b56d1948422dda924fcf450560591e8a19
  Author: Krzysztof Parzyszek <kparzysz at quicinc.com>
  Date:   2023-06-09 (Fri, 09 Jun 2023)

  Changed paths:
    M llvm/include/llvm/CodeGen/RDFGraph.h
    M llvm/lib/CodeGen/RDFGraph.cpp

  Log Message:
  -----------
  [RDF] Define short type names: NodeAddr<XyzNode*> -> Xyz, NFC


  Commit: d3b34b7f3a7cbfc96aea897419f167b5ee19e61a
      https://github.com/llvm/llvm-project/commit/d3b34b7f3a7cbfc96aea897419f167b5ee19e61a
  Author: Krzysztof Parzyszek <kparzysz at quicinc.com>
  Date:   2023-06-09 (Fri, 09 Jun 2023)

  Changed paths:
    M llvm/include/llvm/CodeGen/RDFGraph.h
    M llvm/lib/CodeGen/RDFGraph.cpp

  Log Message:
  -----------
  [RDF] Use trailing return type syntax, NFC

The name rdf::Use conflicts with llvm::Use when both namespaces are
used via `using namespace`. Specifically this happened in the declaration
of DataFlowGraph::newUse (in RDFGraph.cpp):
```
using namespace rdf;

Use newUse(...);  <-- Lookup conflict for "Use"
```
Since the TRT lookup starts in a different namespace than that of the
leading type, this serves as a workaround. In general the rdf namespace
will not likely be introduced via `using namespace`, so this shouldn't
be a problem elsewhere.


Compare: https://github.com/llvm/llvm-project/compare/1c2c66884669...d3b34b7f3a7c