[PATCH] D17241: [Refactor] Move isl_ctx into Scop.

Mon Feb 15 07:45:51 PST 2016

etherzhhb added a comment.

In http://reviews.llvm.org/D17241#352759, @llvm-commits wrote:

> > And if  ScopInfo, DependencyInfo, IslAst are not free in order, it become a total mess ....
>
>
> The pass manager should free IslAst, DependencyInfo, ... __before__ ScopInfo
>  because they have a dependence on the ScopInfo results. Am I wrong?

Sorry I am not precise enough. It should be Scop (the result of ScopInfo), Dependences (the result of DependenceInfo) and IslAst (the result of IslAstInfo).

Let me do some experiments to confirm the order with the debug-pass option.

Repository:
  rL LLVM

http://reviews.llvm.org/D17241