[PATCH] D70726: [OpenMP50] Add parallel master construct

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 5 09:46:05 PST 2019


ABataev added inline comments.


================
Comment at: clang/test/OpenMP/parallel_master_codegen.cpp:67
+
+// CK2:       define internal {{.*}}void [[OMP_OUTLINED]](i32* noalias %.global_tid., i32* noalias %.bound_tid.)
+// CK2-NOT:   __kmpc_global_thread_num
----------------
Not the best idea to use `%.global_tid.` and `%.bound_tid.` or similar швы directly in the test, they may be different on different builds. Better to use regular expressions everywhere in the test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70726





More information about the cfe-commits mailing list