This revision was automatically updated to reflect the committed changes.
Closed by commit rG832cd749131b: [AArch64] Armv8.6-a Matrix Mult Assembly +
Intrinsics (authored by LukeGeeson).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77871/new/
kmclaughlin accepted this revision.
kmclaughlin added a comment.
This revision is now accepted and ready to land.
Thanks for the updates, @LukeGeeson, LGTM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77871/new/
https://reviews.llvm.org/D77871
LukeGeeson updated this revision to Diff 259327.
LukeGeeson marked 5 inline comments as done.
LukeGeeson added a comment.
- fixed typos
- added sroa as mem2reg arg to reduce redundant mem accesses in tests,
refactored test
- addressed other comments
CHANGES SINCE LAST ACTION
kmclaughlin added inline comments.
Comment at: clang/test/CodeGen/aarch64-v8.6a-neon-intrinsics.c:3
+// RUN: -fallow-half-arguments-and-returns -S -disable-O0-optnone -emit-llvm
-o - %s \
+// RUN: | opt -S -mem2reg \
+// RUN: | FileCheck %s
Is it possible to
LukeGeeson added a comment.
Removed reliance on parent revision, harbormaster now builds with unit tests
passing
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77871/new/
https://reviews.llvm.org/D77871
LukeGeeson created this revision.
LukeGeeson added reviewers: ostannard, t.p.northover.
Herald added subscribers: cfe-commits, danielkiss, hiraditya, kristof.beyls.
Herald added a reviewer: rengolin.
Herald added a project: clang.
LukeGeeson added a parent revision: D77540: [PATCH] [ARM]:
