From aad15d812e534b97a3ac5489e933f07b27df53b9 Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Sun, 20 Oct 2019 12:23:38 -0700 Subject: [PATCH] [DRR] Address GCC warning by wrapping for statement body with {} Otherwise, we'll see the following warning when compiling with GCC 8: warning: this ?for? clause does not guard... [-Wmisleading-indentation] PiperOrigin-RevId: 275735925 --- mlir/tools/mlir-tblgen/RewriterGen.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/mlir/tools/mlir-tblgen/RewriterGen.cpp b/mlir/tools/mlir-tblgen/RewriterGen.cpp index cefd1d6..684e619 100644 --- a/mlir/tools/mlir-tblgen/RewriterGen.cpp +++ b/mlir/tools/mlir-tblgen/RewriterGen.cpp @@ -556,7 +556,8 @@ void PatternEmitter::emitRewriteLogic() { os.indent(4) << "\n"; // Resolve each symbol for all range use so that we can loop over them. os << symbolInfoMap.getAllRangeUse( - val, " for (auto *v : {0}) tblgen_values.push_back(v);", "\n"); + val, " for (auto *v : {0}) {{ tblgen_values.push_back(v); }", + "\n"); } os.indent(4) << "\n"; os.indent(4) << "rewriter.replaceOp(op0, tblgen_values);\n"; -- 2.7.4