[llvm] [OpenMP] Move KernelInfoState and AAKernelInfo to OpenMPOpt.h (PR #71878)

via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 13 09:53:03 PST 2023


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 51af040b22c117e4f2429277c515299cd62c58ea f8ae5184867db8cf2776165c0697bd4fd33c1c31 -- llvm/include/llvm/Transforms/IPO/OpenMPOpt.h llvm/lib/Transforms/IPO/OpenMPOpt.cpp
``````````

</details>

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

``````````diff
diff --git a/llvm/include/llvm/Transforms/IPO/OpenMPOpt.h b/llvm/include/llvm/Transforms/IPO/OpenMPOpt.h
index 04d40ab004..5bef03c0a1 100644
--- a/llvm/include/llvm/Transforms/IPO/OpenMPOpt.h
+++ b/llvm/include/llvm/Transforms/IPO/OpenMPOpt.h
@@ -262,10 +262,9 @@ struct AAKernelInfo : public StateWrapper<KernelInfoState, AbstractAttribute> {
   /// Public getter for ReachingKernelEntries
   virtual BooleanStateWithPtrSetVector<Function, false>
   getReachingKernels() = 0;
-  
+
   /// Create an abstract attribute biew for the position \p IRP.
-  static AAKernelInfo &createForPosition(const IRPosition &IRP,
-                                             Attributor &A);
+  static AAKernelInfo &createForPosition(const IRPosition &IRP, Attributor &A);
 
   /// This function should return true if the type of the \p AA is AAKernelInfo
   static bool classof(const AbstractAttribute *AA) {
@@ -279,7 +278,6 @@ struct AAKernelInfo : public StateWrapper<KernelInfoState, AbstractAttribute> {
   const char *getIdAddr() const override { return &ID; }
 
   static const char ID;
-
 };
 
 } // end namespace llvm
diff --git a/llvm/lib/Transforms/IPO/OpenMPOpt.cpp b/llvm/lib/Transforms/IPO/OpenMPOpt.cpp
index 1cf89d63c4..0494b2d717 100644
--- a/llvm/lib/Transforms/IPO/OpenMPOpt.cpp
+++ b/llvm/lib/Transforms/IPO/OpenMPOpt.cpp
@@ -5110,7 +5110,7 @@ private:
 
     for (Kernel K : CallerKernelInfoAA->ReachingKernelEntries) {
       auto *AA = A.getAAFor<AAKernelInfo>(*this, IRPosition::function(*K),
-                                              DepClassTy::REQUIRED);
+                                          DepClassTy::REQUIRED);
 
       if (!AA || !AA->isValidState()) {
         SimplifiedValue = nullptr;
@@ -5182,7 +5182,7 @@ private:
     unsigned AssumedNonSPMDCount = 0, KnownNonSPMDCount = 0;
     for (Kernel K : CallerKernelInfoAA->ReachingKernelEntries) {
       auto *AA = A.getAAFor<AAKernelInfo>(*this, IRPosition::function(*K),
-                                              DepClassTy::REQUIRED);
+                                          DepClassTy::REQUIRED);
       if (!AA || !AA->SPMDCompatibilityTracker.isValidState())
         return indicatePessimisticFixpoint();
 
@@ -5464,7 +5464,7 @@ AAHeapToShared &AAHeapToShared::createForPosition(const IRPosition &IRP,
 }
 
 AAKernelInfo &AAKernelInfo::createForPosition(const IRPosition &IRP,
-                                                      Attributor &A) {
+                                              Attributor &A) {
   AAKernelInfo *AA = nullptr;
   switch (IRP.getPositionKind()) {
   case IRPosition::IRP_INVALID:

``````````

</details>


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


More information about the llvm-commits mailing list