[PATCH] D155514: [BOLT] Introduce ProfileUseDFS option

[Sergey Pupyrev via Phabricator via llvm-commits]

spupyrev added inline comments.


================
Comment at: bolt/lib/Profile/YAMLProfileReader.cpp:109
 
-  BinaryFunction::BasicBlockOrderType DFSOrder = BF.dfs();
+  SmallVector<BinaryBasicBlock *, 0> Order;
+  if (opts::ProfileUseDFS)
----------------
Is there a reason why the type is not `BinaryFunction::BasicBlockOrderType` but an explicit SmallVector?

Also I'm not sure if there is any advantage here to use SmallVector over std::vector, as the order is always non-empty.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155514