[PATCH] D63389: [IDF] Generalize IDFCalculator to be used with Clang's CFG

Mon Jun 17 08:17:19 PDT 2019

kuhar added a comment.

Looks great, I'm only not sure about the GraphDiff removal.

> Practically revert D50675 <https://reviews.llvm.org/D50675>. This was done because GraphDiff is only specialized for BasicBlock, and wouldn't be used for clang::CFGBlcok. Interestingly, it wasn't even used at all: In D45299 <https://reviews.llvm.org/D45299>, it was used at a certain point (https://reviews.llvm.org/D45299?id=160698) but was ultimately removed. Is this okay?

@asbirlea?

Repository:
  rL LLVM

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

https://reviews.llvm.org/D63389