Skip to content

Commit

Permalink
[Polly] Move SimplifyVisitor into polly namespace.
Browse files Browse the repository at this point in the history
Declarations in headers should not be in the anonymous
namespace. Compilers also warn about the use of
<anon namespace>::SimplifyVisitor as a public field in
polly::SimplifyPass and polly::SimplifyPrinterPass.
  • Loading branch information
Meinersbur committed Nov 17, 2020
1 parent b16e4d3 commit fc115f2
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 2 additions & 2 deletions polly/include/polly/Simplify.h
Expand Up @@ -22,7 +22,7 @@ class PassRegistry;
class Pass;
} // namespace llvm

namespace {
namespace polly {
class SimplifyVisitor {
private:
/// The invocation id (if there are multiple instances in the pass manager's
Expand Down Expand Up @@ -114,7 +114,7 @@ class SimplifyVisitor {

void releaseMemory();
};
} // namespace
} // namespace polly

namespace polly {

Expand Down
2 changes: 2 additions & 0 deletions polly/lib/Transform/Simplify.cpp
Expand Up @@ -116,6 +116,7 @@ static isl::union_map underapproximatedAddMap(isl::union_map UMap,

return UResult;
}
} // namespace

/// Return whether at least one simplification has been applied.
bool SimplifyVisitor::isModified() const {
Expand Down Expand Up @@ -678,6 +679,7 @@ void SimplifyVisitor::releaseMemory() {
StmtsRemoved = 0;
}

namespace {
class SimplifyLegacyPass : public ScopPass {
public:
static char ID;
Expand Down

0 comments on commit fc115f2

Please sign in to comment.