[PATCH] D23916: [Polly] Check validity of new access relations. NFC.
Roman via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 26 06:54:23 PDT 2016
gareevroman added inline comments.
================
Comment at: include/polly/Support/ScopHelper.h:451
@@ -436,1 +450,3 @@
+bool isl_space_is_compatible(__isl_keep isl_space *Space1,
+ __isl_keep isl_space *Space2);
} // namespace polly
----------------
gareevroman wrote:
> grosser wrote:
> > This really belongs into isl. We can probably add this here for now, as having a local copy has minimal cost, but we should probably contribute a corresponding patch upstream.
> We should possibly rename it to, for example, SpaceIsCompatible, if we want to declare it here.
"spaceIsCompatible"
https://reviews.llvm.org/D23916
More information about the llvm-commits
mailing list