[PATCH] D60568: [OpenMP] Add support for registering requires directives with the runtime
Gheorghe-Teodor Bercea via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 17 11:44:24 PDT 2019
gtbercea marked 2 inline comments as done.
gtbercea added inline comments.
Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:8975
+ "Target region emitted before requires directive.");
+ HasRequiresUnifiedSharedMemory = true;
> The message speaks about target region but points to the clause. You need to correct the message.
"A target region was emitted before this requires directive." ?
CHANGES SINCE LAST ACTION
More information about the cfe-commits