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: > 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. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60568/new/ https://reviews.llvm.org/D60568 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
