[PATCH] D57113: [clang-tidy] openmp-use-default-none - a new module and a check

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 25 06:10:09 PST 2019


ABataev added inline comments.


================
Comment at: test/clang-tidy/openmp-use-default-none.cpp:26
+void t2() {
+#pragma omp parallel default(none)
+  ;
----------------
JonasToth wrote:
> AFAIK `default(private)` should exist as well, please add tests for the other kinds, too.
No, only `default(none|shared)` exists


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D57113





More information about the cfe-commits mailing list