[PATCH] D78861: [Attributor] [WIP] Track AA dependency using dependency graph

[Luofan Chen via Phabricator via llvm-commits]

bbn added a comment.

In D78861#2081311 <https://reviews.llvm.org/D78861#2081311>, @kuter wrote:

> This does not compile for me. The compiler error that I get is about creating a `GraphTraits` specialization outside of the `llvm` namespace.
>  When i put the `GraphTraits` in llvm namespace it does compile.


What do you mean by "put the `GraphTraits` in llvm namespace"?

> But when I run it I get a segfault.

Yes, I also got this segfault when running this on assign.ll. I think this issue is related to the `AbstractAttribute::print()` function and is not caused by the
dependency graph. I am currently looking into this and I will write a new print function for AA.


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

https://reviews.llvm.org/D78861