[PATCH] ADT Fixups

Patrick_Alexander_Si · November 16, 2009, 6:09pm

Hello,

I posted an earlier version of this patch some months ago and realized I've still been sitting on it. The earlier version of this patch was rejected for adding a cycle detection function to SCCIterator.h. This function is now removed.

This patch makes the following changes:
1. Allow SCCIterator to work with GraphT types that are constant.
2. Allow SCCIterator to work with inverse graphs.
3. Fix an incorrect comment in GraphTraits.h (the type in the comment was given as GraphType* when it is actually const GraphType &).

I think these changes should be pretty uncontroversial. Would someone with commit authority please apply this for me?

--Patrick

Index: include/llvm/ADT/SCCIterator.h

Duncan_Sands · November 17, 2009, 10:55am

Hi Patrick,

I think these changes should be pretty uncontroversial. Would someone with commit authority please apply this for me?

thanks for the patch. Applied in commit 89091.

Ciao,

Duncan.

Patrick_Alexander_Si · April 30, 2010, 4:21pm

Would someone please merge commit 89091 from the mainline into release_26?

Thanks,
--Patrick

Duncan Sands wrote:

To_ro_k_Edwin · April 30, 2010, 4:24pm

I don't think the release_* branches are ever touched after a release.

Best regards,
--Edwin