[llvm] b8ff08d - [Frontend][OpenMP] Add missing "return" statement after 40137ff0d81be

Krzysztof Parzyszek via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 22 13:35:49 PDT 2024


Author: Krzysztof Parzyszek
Date: 2024-04-22T15:35:42-05:00
New Revision: b8ff08d0e668e5397dd799b76ede0bd54fcba75c

URL: https://github.com/llvm/llvm-project/commit/b8ff08d0e668e5397dd799b76ede0bd54fcba75c
DIFF: https://github.com/llvm/llvm-project/commit/b8ff08d0e668e5397dd799b76ede0bd54fcba75c.diff

LOG: [Frontend][OpenMP] Add missing "return" statement after 40137ff0d81be

When responding to review comments, `return {}` was accidentally replaced
by `std::nullptr` instead of `return std::nullptr`.

Added: 
    

Modified: 
    llvm/lib/Frontend/OpenMP/OMP.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Frontend/OpenMP/OMP.cpp b/llvm/lib/Frontend/OpenMP/OMP.cpp
index e958bced3a4224..e097510592c9ab 100644
--- a/llvm/lib/Frontend/OpenMP/OMP.cpp
+++ b/llvm/lib/Frontend/OpenMP/OMP.cpp
@@ -29,7 +29,7 @@ namespace llvm::omp {
 ArrayRef<Directive> getLeafConstructs(Directive D) {
   auto Idx = static_cast<std::size_t>(D);
   if (Idx >= Directive_enumSize)
-    std::nullopt;
+    return std::nullopt;
   const auto *Row = LeafConstructTable[LeafConstructTableOrdering[Idx]];
   return ArrayRef(&Row[2], static_cast<int>(Row[1]));
 }


        


More information about the llvm-commits mailing list