Meinersbur accepted this revision. Meinersbur added a comment. This revision is now accepted and ready to land.
LGTM ================ Comment at: lib/Sema/SemaStmtAttr.cpp:183 {nullptr, nullptr}}; - for (const auto *I : Attrs) { ---------------- [nit] unrelated whitespace change? ================ Comment at: test/CodeGenCXX/pragma-unroll-and-jam.cpp:4 +void unroll_and_jam(int *List, int Length, int Value) { + // CHECK: define {{.*}} @_Z14unroll_and_jam +#pragma unroll_and_jam ---------------- [suggestion] `CHECK-LABEL`? (applies to the function names below as well) https://reviews.llvm.org/D47267 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits