tianshilei1992 added inline comments.

================
Comment at: clang/test/OpenMP/atomic_ast_print.cpp:35
   }
+// omp50-error@+4 {{unexpected OpenMP clause 'compare' in directive '#pragma 
omp atomic'}}
+// omp50-note@+4 {{expected an expression statement}}
----------------
ABataev wrote:
> ast print tests are usually successful, better to add/modify special tests 
> for incorrect cases (the ones ending with _messages.cpp)
Yeah, I'll also do that.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116637/new/

https://reviews.llvm.org/D116637

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to