[PATCH] D51331: [OPENMP] Create non-const ident_t structs.

2018-08-29 Thread Mike Rice via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL340934: [OPENMP] Create non-const ident_t objects. (authored by mikerice, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D51331?vs=162776&id=1

[PATCH] D51331: [OPENMP] Create non-const ident_t structs.

2018-08-29 Thread Mike Rice via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC340934: [OPENMP] Create non-const ident_t objects. (authored by mikerice, committed by ). Repository: rC Clang https://reviews.llvm.org/D51331 Files: lib/CodeGen/CGOpenMPRuntime.cpp test/OpenMP/di

[PATCH] D51331: [OPENMP] Create non-const ident_t structs.

2018-08-29 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev accepted this revision. ABataev added a comment. This revision is now accepted and ready to land. LG https://reviews.llvm.org/D51331 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c

[PATCH] D51331: [OPENMP] Create non-const ident_t structs.

2018-08-29 Thread Andrey Churbanov via Phabricator via cfe-commits
AndreyChurbanov added a comment. In https://reviews.llvm.org/D51331#1216509, @ABataev wrote: > When is ITT Notify used? Does it have some preconditions like debug info, > some optimizations level etc.? The libittnotify library is used by Intel tools (VTune Amplifier, Inspector, Spatial Adviso

[PATCH] D51331: [OPENMP] Create non-const ident_t structs.

2018-08-28 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev added a comment. When is ITT Notify used? Does it have some preconditions like debug info, some optimizations level etc.? https://reviews.llvm.org/D51331 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/

[PATCH] D51331: [OPENMP] Create non-const ident_t structs.

2018-08-27 Thread Mike Rice via Phabricator via cfe-commits
mikerice created this revision. mikerice added a reviewer: ABataev. Herald added subscribers: jfb, guansong. ITT Notify in the OpenMP runtime uses (and writes to) the reserved_2 field in the ident_t struct. See __kmp_itt_region_forking in openmp/runtime/src/kmp_itt.inl. Currently clang is crea