gtbercea marked an inline comment as done. gtbercea added inline comments.
================ Comment at: lib/CodeGen/CGOpenMPRuntime.h:644 + /// Flag for keeping track of weather a target region has been emitted. + bool hasEmittedTargetRegion = false; + ---------------- ABataev wrote: > Why do you need this? I think your function should be called without any > conditions. It does not depend on the presence of the target regions or not. > Plus, your check is not consistent if you're calling the function from > another module, which has target region inside. This does not determine if the function is called or not. This helps determine the flags with which the function is called. 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