[PATCH] D47438: [ScopInfo] Update Scop::addUserContext() to C++ interface
SAHIL GIRISH YERAWAR via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun May 27 23:00:20 PDT 2018
cs15btech11044 created this revision.
cs15btech11044 added reviewers: grosser, bollu, philip.pfaffe, chelini, Meinersbur.
Herald added a subscriber: llvm-commits.
This patch updates `Scop::addUserContext()` function to the new C++ interface.
Repository:
rPLO Polly
https://reviews.llvm.org/D47438
Files:
lib/Analysis/ScopInfo.cpp
Index: lib/Analysis/ScopInfo.cpp
===================================================================
--- lib/Analysis/ScopInfo.cpp
+++ lib/Analysis/ScopInfo.cpp
@@ -2112,49 +2112,38 @@
if (UserContextStr.empty())
return;
- isl_set *UserContext =
- isl_set_read_from_str(getIslCtx().get(), UserContextStr.c_str());
- isl_space *Space = getParamSpace().release();
- if (isl_space_dim(Space, isl_dim_param) !=
- isl_set_dim(UserContext, isl_dim_param)) {
- auto SpaceStr = isl_space_to_str(Space);
+ isl::set UserContext = isl::set(getIslCtx(), UserContextStr.c_str());
+ isl::space Space = isl::manage(getParamSpace().release());
+ if (Space.dim(isl::dim::param) != UserContext.dim(isl::dim::param)) {
+ auto SpaceStr = Space.to_str();
errs() << "Error: the context provided in -polly-context has not the same "
<< "number of dimensions than the computed context. Due to this "
<< "mismatch, the -polly-context option is ignored. Please provide "
<< "the context in the parameter space: " << SpaceStr << ".\n";
- free(SpaceStr);
- isl_set_free(UserContext);
- isl_space_free(Space);
return;
}
- for (unsigned i = 0; i < isl_space_dim(Space, isl_dim_param); i++) {
+ for (unsigned i = 0; i < Space.dim(isl::dim::param); i++) {
std::string NameContext = Context.get_dim_name(isl::dim::param, i);
- std::string NameUserContext =
- isl_set_get_dim_name(UserContext, isl_dim_param, i);
+ std::string NameUserContext = UserContext.get_dim_name(isl::dim::param, i);
if (NameContext != NameUserContext) {
- auto SpaceStr = isl_space_to_str(Space);
+ auto SpaceStr = Space.to_str();
errs() << "Error: the name of dimension " << i
<< " provided in -polly-context "
<< "is '" << NameUserContext << "', but the name in the computed "
<< "context is '" << NameContext
<< "'. Due to this name mismatch, "
<< "the -polly-context option is ignored. Please provide "
<< "the context in the parameter space: " << SpaceStr << ".\n";
- free(SpaceStr);
- isl_set_free(UserContext);
- isl_space_free(Space);
return;
}
- UserContext =
- isl_set_set_dim_id(UserContext, isl_dim_param, i,
- isl_space_get_dim_id(Space, isl_dim_param, i));
+ UserContext = UserContext.set_dim_id(isl::dim::param, i,
+ Space.get_dim_id(isl::dim::param, i));
}
- Context = Context.intersect(isl::manage(UserContext));
- isl_space_free(Space);
+ Context = Context.intersect(UserContext);
}
void Scop::buildInvariantEquivalenceClasses() {
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D47438.148776.patch
Type: text/x-patch
Size: 2730 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180528/8a679c4a/attachment.bin>
More information about the llvm-commits
mailing list