[PATCH] D35471: [Polly] [RFC] Calculate AST expression type

Maximilian Falkenstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 17 16:38:46 PDT 2017


maxf added a comment.

No, this is not supposed to only change isl inside of polly. Among the things that were noted when I originally wanted to upstream the code into isl is a lack of motivation for the interface, since at that time it was a big patch to isl only. After discussion with Sven, we agreed on changing the original interface (store information in AST (expression) nodes instead of directly inside expressions) but before cleaning up and preparing another round of patches and resubmitting it seems like a good idea to show the whole code in one place so that we can agree on what and how we'd like isl to provide.


https://reviews.llvm.org/D35471





More information about the llvm-commits mailing list