[PATCH] D17241: [Refactor] Use ManagedStatic to manage the global isl_ctx.

[Johannes Doerfert via llvm-commits]

jdoerfert added a subscriber: jdoerfert.
jdoerfert added a comment.

I think this is not what we want. Globals are always bad when we think about parallel execution of passes. While we __still__ have a global map flying around that nobody cares to remove we... we should not introduce new globals that will break the assumption that non-connected parts of the module can be compiled concurrently. As an alternative I would suggest to move the isl_ctx into the Scop class and let it deal correctly with deallocating all derived objects once it (the Scop) is deleted.


Repository:
  rL LLVM

http://reviews.llvm.org/D17241