[PATCH] D139166: [OPENMP51] Codegen support for error directive.

Jennifer Yu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 2 17:19:08 PST 2022


jyu2 added inline comments.


================
Comment at: clang/lib/CodeGen/CGOpenMPRuntime.h:334
   llvm::Value *emitUpdateLocation(CodeGenFunction &CGF, SourceLocation Loc,
-                                  unsigned Flags = 0);
+                                  unsigned Flags = 0, bool EmitLoc = false);
 
----------------
jyu2 wrote:
> jdoerfert wrote:
> > Why the new argument?
> Emit error without -g (debug info turn on.)
Hi @jdoerfert, 
Thanks for your review.  I added comment on what meaning of  EmitLoc as:

/// \param EmitLoc emit source location with debug-info is off.

Do you mean that is not clear to you?  Any suggestion?

Thanks.
Jennifer



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139166/new/

https://reviews.llvm.org/D139166



More information about the cfe-commits mailing list