This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[PATCH][ARM] Add source mode to coprocessor pattern SETs
- From: Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 10 Apr 2017 12:17:27 +0100
- Subject: [PATCH][ARM] Add source mode to coprocessor pattern SETs
- Authentication-results: sourceware.org; auth=none
Hi all,
Something I've missed during the review of these coprocessor intrinsics patterns is that the SET source is missing a mode.
So we get warnings during the build process.
This patch just trivially adds the appropriate modes to silence the warning.
Bootstrapped and tested on arm-none-linux-gnueabihf.
Committing as obvious.
Thanks,
Kyrill
2016-04-10 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
* config/arm/arm.md (<mrc>): Add mode to SET source.
(<mrrc>): Likewise.
commit b010fcbdd4276a6dbd861e31ca7305c1c57e04ce
Author: Kyrylo Tkachov <kyrylo.tkachov@arm.com>
Date: Thu Jan 19 14:16:23 2017 +0000
[ARM] Add source mode to coprocessor pattern SETs
diff --git a/gcc/config/arm/arm.md b/gcc/config/arm/arm.md
index 2251991..8f95a68 100644
--- a/gcc/config/arm/arm.md
+++ b/gcc/config/arm/arm.md
@@ -12050,7 +12050,7 @@ (define_insn "<mcr>"
(define_insn "<mrc>"
[(set (match_operand:SI 0 "s_register_operand" "=r")
- (unspec_volatile [(match_operand:SI 1 "immediate_operand" "n")
+ (unspec_volatile:SI [(match_operand:SI 1 "immediate_operand" "n")
(match_operand:SI 2 "immediate_operand" "n")
(match_operand:SI 3 "immediate_operand" "n")
(match_operand:SI 4 "immediate_operand" "n")
@@ -12085,7 +12085,7 @@ (define_insn "<mcrr>"
(define_insn "<mrrc>"
[(set (match_operand:DI 0 "s_register_operand" "=r")
- (unspec_volatile [(match_operand:SI 1 "immediate_operand" "n")
+ (unspec_volatile:DI [(match_operand:SI 1 "immediate_operand" "n")
(match_operand:SI 2 "immediate_operand" "n")
(match_operand:SI 3 "immediate_operand" "n")] MRRCI))]
"arm_coproc_builtin_available (VUNSPEC_<MRRC>)"