This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGf225898a7c61: [Clang][AArch64][SME] Add intrinsics for ZA
array load/store (LDR/STR) (authored by bryanpkc).
Repository:
rG LLVM Github Monorepo
sdesmalen accepted this revision.
sdesmalen added a comment.
This revision is now accepted and ready to land.
Thanks for the quick fix!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134678/new/
https://reviews.llvm.org/D134678
bryanpkc updated this revision to Diff 541132.
bryanpkc marked an inline comment as done.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134678/new/
https://reviews.llvm.org/D134678
Files:
clang/include/clang/Basic/arm_sme.td
bryanpkc marked an inline comment as done.
bryanpkc added inline comments.
Comment at: clang/test/CodeGen/aarch64-sme-intrinsics/acle_sme_str.c:24
+// CHECK-NEXT:[[TMP0:%.*]] = getelementptr i8, ptr [[PTR:%.*]], i64
[[MULVL]]
+// CHECK-NEXT:tail call void
sdesmalen added inline comments.
Comment at: clang/test/CodeGen/aarch64-sme-intrinsics/acle_sme_str.c:24
+// CHECK-NEXT:[[TMP0:%.*]] = getelementptr i8, ptr [[PTR:%.*]], i64
[[MULVL]]
+// CHECK-NEXT:tail call void @llvm.aarch64.sme.str(i32 [[SLICE_BASE:%.*]],
ptr
bryanpkc updated this revision to Diff 540839.
bryanpkc added a comment.
Removed the attribute macro from tests, as @sdesmalen suggested.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134678/new/
https://reviews.llvm.org/D134678
Files:
bryanpkc updated this revision to Diff 531953.
bryanpkc added a comment.
Also added SME attributes to the tests.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134678/new/
https://reviews.llvm.org/D134678
Files:
bryanpkc updated this revision to Diff 531949.
bryanpkc added a comment.
Instead of defining new type flags, `EmitAArch64SMEBuiltinExpr` is updated to
switch on `BuiltinID`, as suggested by @sdesmalen.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
bryanpkc updated this revision to Diff 526361.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134678/new/
https://reviews.llvm.org/D134678
Files:
clang/include/clang/Basic/TargetBuiltins.h
clang/include/clang/Basic/arm_sme.td
bryanpkc updated this revision to Diff 492679.
bryanpkc retitled this revision from "[Clang][AArch64] Add SME ldr and str
intrinsic" to "[Clang][AArch64][SME] Add intrinsics for ZA array load/store
(LDR/STR)".
bryanpkc edited the summary of this revision.
bryanpkc added a comment.
Rebased and
10 matches
Mail list logo