[PATCH] D155849: [NFC][clang] Fix static analyzer concerns

Tom Honermann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 20 09:49:12 PDT 2023


tahonermann accepted this revision.
tahonermann added a comment.

Looks good; the default implementations would definitely do the wrong thing, so this is a good find. My suggested edit is just to add a blank line.



================
Comment at: clang/lib/CodeGen/CGStmtOpenMP.cpp:1823-1826
+  OMPTransformDirectiveScopeRAII(const OMPTransformDirectiveScopeRAII &) =
+      delete;
+  OMPTransformDirectiveScopeRAII &
+  operator=(const OMPTransformDirectiveScopeRAII &) = delete;
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155849



More information about the cfe-commits mailing list