curdeius accepted this revision. curdeius added a comment. This revision is now accepted and ready to land.
LGTM with a single comment on the test. ================ Comment at: clang/unittests/Format/FormatTest.cpp:17676-17684 + format("void foo() {\n" + "#if 1\n" + " #pragma omp simd\n" + " for (int k = 0; k < 10; k++) {\n" + " #pragma omp simd\n" + " for (int j = 0; j < 10; j++) {\n" + " }\n" ---------------- Last nit, I'd mess up spaces like suggested here. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92753/new/ https://reviews.llvm.org/D92753 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits