Skip to content

Users/tsitdikov #82757

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Feb 23, 2024
Merged

Users/tsitdikov #82757

merged 2 commits into from
Feb 23, 2024

Conversation

tsitdikov
Copy link
Contributor

Fix Test ARM SME library and build rule.

@llvmbot
Copy link
Member

llvmbot commented Feb 23, 2024

@llvm/pr-subscribers-mlir

@llvm/pr-subscribers-mlir-sme

Author: None (tsitdikov)

Changes

Fix Test ARM SME library and build rule.


Full diff: https://github.com/llvm/llvm-project/pull/82757.diff

2 Files Affected:

  • (modified) mlir/test/lib/Dialect/ArmSME/CMakeLists.txt (+2)
  • (modified) utils/bazel/llvm-project-overlay/mlir/test/BUILD.bazel (+2)
diff --git a/mlir/test/lib/Dialect/ArmSME/CMakeLists.txt b/mlir/test/lib/Dialect/ArmSME/CMakeLists.txt
index de4971ff7eb3d1..e942c7b8ac058c 100644
--- a/mlir/test/lib/Dialect/ArmSME/CMakeLists.txt
+++ b/mlir/test/lib/Dialect/ArmSME/CMakeLists.txt
@@ -8,6 +8,8 @@ add_mlir_library(MLIRArmSMETestPasses
   MLIRArithToArmSME
   MLIRArmSMEToLLVM
   MLIRArmSMEToSCF
+  MLIRArmSMETransforms
+  MLIRArmSVETransforms
   MLIRIR
   MLIRPass
   MLIRTransforms
diff --git a/utils/bazel/llvm-project-overlay/mlir/test/BUILD.bazel b/utils/bazel/llvm-project-overlay/mlir/test/BUILD.bazel
index 497256573dfc5a..68d9b23fd5643e 100644
--- a/utils/bazel/llvm-project-overlay/mlir/test/BUILD.bazel
+++ b/utils/bazel/llvm-project-overlay/mlir/test/BUILD.bazel
@@ -914,6 +914,8 @@ cc_library(
         "//mlir:ArithToArmSME",
         "//mlir:ArmSMEToLLVM",
         "//mlir:ArmSMEToSCF",
+        "//mlir:ArmSMETransforms",
+        "//mlir:ArmSVETransforms",
         "//mlir:IR",
         "//mlir:Pass",
         "//mlir:Transforms",

@tsitdikov tsitdikov merged commit e09e0d5 into main Feb 23, 2024
@tsitdikov tsitdikov deleted the users/tsitdikov branch February 23, 2024 12:54
@joker-eph
Copy link
Collaborator

Please write correct commit message / title (@akuegel : this is part of the approval / review).

@akuegel
Copy link
Member

akuegel commented Feb 24, 2024

Please write correct commit message / title (@akuegel : this is part of the approval / review).

Yes, sorry, I only checked the code change and missed this. I saw it after it was committed, and already told the author. But I guess this cannot be changed afterwards.

@joker-eph
Copy link
Collaborator

No big deal, if you noticed it's good, I just pointed it for next time :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants