chandlerc added inline comments.
================
Comment at: llvm/lib/Target/X86/ImmutableGraph.h:46-56
+ using NodeValueT = _NodeValueT;
+ using EdgeValueT = _EdgeValueT;
+ using size_type = _SizeT;
+ class Node;
+ class Edge {
+ friend class ImmutableGraph;
+ template <typename> friend class ImmutableGraphBuilder;
----------------
Folks, this isn't even close to following LLVM's coding conventions or naming
conventions.
These violate the C++ standard.
This shouldn't have been landed as-is. Can you all back this out and actually
dig into the review and get this to match LLVM's actual coding style and
standards?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75936/new/
https://reviews.llvm.org/D75936
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
