[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
Mon Apr 15 08:44:18 PDT 2019
gtbercea marked 2 inline comments as done.
gtbercea added inline comments.
Comment at: lib/CodeGen/CGOpenMPRuntime.h:1438
+ /// requires directives was used in the current module.
+ virtual llvm::Function *emitRequiresDirectiveRegFun();
> ABataev wrote:
> > Why do you need this vertual funtion? I think static local is going to be enough
> Can you make it `const` member function?
Cannot add that due to createRuntimeFunction not being const.
CHANGES SINCE LAST ACTION
More information about the cfe-commits