[PATCH] D64088: [DDG] DirectedGraph as a base class for various dependence graphs such as DDG and PDG.

Fri Jul 12 10:24:09 PDT 2019

bmahjour added a comment.

In D64088#1579497 <https://reviews.llvm.org/D64088#1579497>, @fhahn wrote:

> In D64088#1579463 <https://reviews.llvm.org/D64088#1579463>, @Meinersbur wrote:
>
> > To not to be stuck in details and not block the dependence graph patches, we maybe should land the patch and work on it in-tree when its users materialize?
>
>
> Sounds good, but I think to land it, it would be good if we have some unit tests for the current implementation, to make sure it builds, we don’t regress and have sanitizer coverage.

Please see the updated patch with tests for various functions of the DirectedGraph plus a GraphTrait instantiation as well as a test of SCC iterator over the directed graph.

Repository:
  rL LLVM

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

https://reviews.llvm.org/D64088