This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[Patch obvious AArch64] Wrap SHIFT_COUNT_TRUNCATED in brackets
- From: James Greenhalgh <james dot greenhalgh at arm dot com>
- To: <gcc-patches at gcc dot gnu dot org>
- Cc: <nd at arm dot com>, <marcus dot shawcroft at arm dot com>
- Date: Tue, 17 May 2016 09:57:41 +0100
- Subject: [Patch obvious AArch64] Wrap SHIFT_COUNT_TRUNCATED in brackets
- Authentication-results: sourceware.org; auth=none
- Nodisclaimer: True
- Spamdiagnosticmetadata: NSPM
- Spamdiagnosticoutput: 1:23
Hi,
This is probably not going to be an issue, but we should wrap the macro
definition in () just in case someone does want to use it with a higher
precedence operator.
Applied as obvious as revision 236312.
Thanks,
James
---
2016-05-16 James Greenhalgh <james.greenhalgh@arm.com>
* config/aarch64/aarch64.c (SHIFT_COUNT_TRUNCATED): Wrap definition
in brackets.
diff --git a/gcc/config/aarch64/aarch64.h b/gcc/config/aarch64/aarch64.h
index 6eb31e9..fa941b6 100644
--- a/gcc/config/aarch64/aarch64.h
+++ b/gcc/config/aarch64/aarch64.h
@@ -835,7 +835,7 @@ typedef struct
#define CANNOT_CHANGE_MODE_CLASS(FROM, TO, CLASS) \
aarch64_cannot_change_mode_class (FROM, TO, CLASS)
-#define SHIFT_COUNT_TRUNCATED !TARGET_SIMD
+#define SHIFT_COUNT_TRUNCATED (!TARGET_SIMD)
/* Choose appropriate mode for caller saves, so we do the minimum
required size of load/store. */