[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