[llvm] [MCA] Do not allocate space for DependenceEdge by default in DependencyGraphNode (NFC) (PR #125080)
Andrea Di Biagio via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 30 10:55:04 PST 2025
================
@@ -228,7 +228,7 @@ class DependencyGraph {
unsigned Depth;
DependencyEdge CriticalPredecessor;
- SmallVector<DependencyEdge, 8> OutgoingEdges;
+ SmallVector<DependencyEdge, 0> OutgoingEdges;
----------------
adibiagio wrote:
I think it is useful to have a comment (not more than a couple of lines) explaining why zero was used for the number of inline elements. Otherwise it would be unclear for the reader why we chose that number in this case.
https://github.com/llvm/llvm-project/pull/125080
More information about the llvm-commits
mailing list