[PATCH] D74993: [AArch64] SVE implies fullfp16
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 21 15:35:04 PST 2020
efriedma created this revision.
efriedma added reviewers: sdesmalen, cameron.mcinally.
Herald added subscribers: hiraditya, kristof.beyls, tschuett.
Herald added a project: LLVM.
This is explicitly guaranteed in ARMARM. And it makes reasoning about vectors easier: we can assume that if a vector operation is legal, the corresponding scalar operation is also legal.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D74993
Files:
llvm/lib/Target/AArch64/AArch64.td
llvm/test/CodeGen/AArch64/sve-vector-splat.ll
Index: llvm/test/CodeGen/AArch64/sve-vector-splat.ll
===================================================================
--- llvm/test/CodeGen/AArch64/sve-vector-splat.ll
+++ llvm/test/CodeGen/AArch64/sve-vector-splat.ll
@@ -1,4 +1,4 @@
-; RUN: llc -mtriple=aarch64-linux-gnu -mattr=+sve,+fullfp16 < %s | FileCheck %s
+; RUN: llc -mtriple=aarch64-linux-gnu -mattr=+sve < %s | FileCheck %s
;; Splats of legal integer vector types
Index: llvm/lib/Target/AArch64/AArch64.td
===================================================================
--- llvm/lib/Target/AArch64/AArch64.td
+++ llvm/lib/Target/AArch64/AArch64.td
@@ -101,7 +101,7 @@
"true", "Enable v8.2 data Cache Clean to Point of Persistence" >;
def FeatureSVE : SubtargetFeature<"sve", "HasSVE", "true",
- "Enable Scalable Vector Extension (SVE) instructions">;
+ "Enable Scalable Vector Extension (SVE) instructions", [FeatureFullFP16]>;
def FeatureSVE2 : SubtargetFeature<"sve2", "HasSVE2", "true",
"Enable Scalable Vector Extension 2 (SVE2) instructions", [FeatureSVE]>;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D74993.246022.patch
Type: text/x-patch
Size: 1054 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200221/70d83cc0/attachment.bin>
More information about the llvm-commits
mailing list