[PATCH] D25506: [LCG] Add utilities to compute parent and ascestor relationships between SCCs.

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 12 02:31:21 PDT 2016


chandlerc created this revision.
chandlerc added a reviewer: sanjoy.
chandlerc added a subscriber: llvm-commits.
Herald added a subscriber: mcrosier.

These will be fairly expensive routines to call and might be abused in
real code, but are quite useful when debugging or in asserts and are
reasonable and well formed properties to query.

I've used one of them in an assert that was requested in a code review
here.

My real question: are these utilities worth adding now that we see the
amount of code involved? I just want to double check that this is what
was expected when it was brought up in code review.

Provided this is what we want, I'll add more comprehensive coverage of
these routines in the unittests to make sure they actually work
correctly, etc. Just wanted the sanity check first.


https://reviews.llvm.org/D25506

Files:
  include/llvm/Analysis/LazyCallGraph.h
  lib/Analysis/LazyCallGraph.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D25506.74346.patch
Type: text/x-patch
Size: 4063 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161012/162c0d27/attachment.bin>


More information about the llvm-commits mailing list