[flang-commits] [flang] [Flang][OpenMP][NFC] Track Objects for BlockArgs (PR #197442)

[via flang-commits]

github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff origin/main HEAD --extensions h,cpp -- flang/include/flang/Support/OpenMP-utils.h flang/lib/Lower/OpenMP/ClauseProcessor.cpp flang/lib/Lower/OpenMP/ClauseProcessor.h flang/lib/Lower/OpenMP/OpenMP.cpp flang/lib/Lower/OpenMP/Utils.cpp flang/lib/Lower/OpenMP/Utils.h --diff_from_common_commit
``````````

:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/flang/include/flang/Support/OpenMP-utils.h b/flang/include/flang/Support/OpenMP-utils.h
index d10659c91..cb3d65441 100644
--- a/flang/include/flang/Support/OpenMP-utils.h
+++ b/flang/include/flang/Support/OpenMP-utils.h
@@ -35,7 +35,8 @@ struct EntryBlockArgsEntry {
   llvm::SmallVector<const Fortran::semantics::Symbol *> getSyms() const {
     llvm::SmallVector<const Fortran::semantics::Symbol *> syms;
     syms.reserve(objects.size());
-    llvm::transform(objects, std::back_inserter(syms), [](const Fortran::lower::omp::Object &object) { return object.sym(); });
+    llvm::transform(objects, std::back_inserter(syms),
+        [](const Fortran::lower::omp::Object &object) { return object.sym(); });
     return syms;
   }
 };
@@ -63,7 +64,10 @@ struct EntryBlockArgs {
     llvm::SmallVector<const semantics::Symbol *> syms;
     auto appendSyms = [&syms](const EntryBlockArgsEntry &entry) {
       syms.reserve(syms.size() + entry.objects.size());
-    llvm::transform(entry.objects, std::back_inserter(syms), [](const Fortran::lower::omp::Object &object) { return object.sym(); });
+      llvm::transform(entry.objects, std::back_inserter(syms),
+          [](const Fortran::lower::omp::Object &object) {
+            return object.sym();
+          });
     };
     appendSyms(hasDeviceAddr);
     appendSyms(inReduction);
diff --git a/flang/lib/Lower/OpenMP/ClauseProcessor.cpp b/flang/lib/Lower/OpenMP/ClauseProcessor.cpp
index 225542df6..213b5f783 100644
--- a/flang/lib/Lower/OpenMP/ClauseProcessor.cpp
+++ b/flang/lib/Lower/OpenMP/ClauseProcessor.cpp
@@ -1552,7 +1552,9 @@ static llvm::SmallVector<const semantics::Symbol *>
 getObjectsSyms(llvm::ArrayRef<Object> objects) {
   llvm::SmallVector<const semantics::Symbol *> syms;
   syms.reserve(objects.size());
-  llvm::transform(objects, std::back_inserter(syms), [](const Fortran::lower::omp::Object &object) { return object.sym(); });
+  llvm::transform(
+      objects, std::back_inserter(syms),
+      [](const Fortran::lower::omp::Object &object) { return object.sym(); });
   return syms;
 }
 
diff --git a/flang/lib/Lower/OpenMP/OpenMP.cpp b/flang/lib/Lower/OpenMP/OpenMP.cpp
index 92dd335c4..28e507fb6 100644
--- a/flang/lib/Lower/OpenMP/OpenMP.cpp
+++ b/flang/lib/Lower/OpenMP/OpenMP.cpp
@@ -78,9 +78,10 @@ static llvm::SmallVector<Object>
 makeObjects(llvm::ArrayRef<const semantics::Symbol *> syms) {
   llvm::SmallVector<Object> objects;
   objects.reserve(syms.size());
-  llvm::transform(syms, std::back_inserter(objects), [](const semantics::Symbol *sym) {
-    return Object{const_cast<semantics::Symbol *>(sym), std::nullopt};
-  });
+  llvm::transform(
+      syms, std::back_inserter(objects), [](const semantics::Symbol *sym) {
+        return Object{const_cast<semantics::Symbol *>(sym), std::nullopt};
+      });
   return objects;
 }
 

``````````

</details>


https://github.com/llvm/llvm-project/pull/197442