[PATCH] D39352: [SimplifyCFG] Don't do if-conversion if there is a long dependence chain

[Guozhi Wei via Phabricator via llvm-commits]

Carrot added inline comments.


================
Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:505
+  // Accumulated latency of each instruction in their BBs.
+  std::map<Instruction *, int> BB1_instructions;
+  std::map<Instruction *, int> BB2_instructions;
----------------
mgrang wrote:
> How big are the keys of your map? Can you use llvm's map containers (like DenseMap, etc)?
The size of the map is BB.size() - 1, and can't be greater than 40.


https://reviews.llvm.org/D39352