[PATCH] D131587: [CodeGen] Deduplicate restore blocks in branch relaxation

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/CodeGen/BranchRelaxation.cpp:66
+    /// and fixupUnconditionalBranch for details.
+    SmallVector<MachineBasicBlock *, 1> RestoreBlocks;
+
----------------
How often are there RestoreBlocks? I wonder if this makes sense to use TinyPtrVector so that we don't enlarge the BasicBlockInfo as much. This SmallVector is probably at least 24 bytes?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131587