public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [Patch ARM] Fix PR target/61413 - definition of __ARM_SIZEOF_WCHAR_T
@ 2014-08-12 15:01 Ramana Radhakrishnan
  0 siblings, 0 replies; only message in thread
From: Ramana Radhakrishnan @ 2014-08-12 15:01 UTC (permalink / raw)
  To: gcc-patches

[-- Attachment #1: Type: text/plain, Size: 457 bytes --]

Hi,

The ACLE 
http://infocenter.arm.com/help/topic/com.arm.doc.ihi0053c/IHI0053C_acle_2_0.pdf 
expects this macro to be defined in terms of byte sizes but we return 
bit sizes *and* additionally do not handle -fshort-wchar.

Applied to trunk.

regards
Ramana

2014-08-12  Ramana Radhakrishnan  <ramana.radhakrishnan@arm.com>

         PR target/61413
         * config/arm/arm.h (TARGET_CPU_CPP_BUILTINS): Fix definition
         of __ARM_SIZEOF_WCHAR_T.


[-- Attachment #2: p1.txt --]
[-- Type: text/plain, Size: 607 bytes --]

diff --git a/gcc/config/arm/arm.h b/gcc/config/arm/arm.h
index 047df97..96e8626 100644
--- a/gcc/config/arm/arm.h
+++ b/gcc/config/arm/arm.h
@@ -74,8 +74,8 @@ extern char arm_arch_name[];
 	builtin_define_with_int_value (				\
 	  "__ARM_SIZEOF_MINIMAL_ENUM",				\
 	  flag_short_enums ? 1 : 4);				\
-	builtin_define_with_int_value (				\
-	  "__ARM_SIZEOF_WCHAR_T", WCHAR_TYPE_SIZE);		\
+	builtin_define_type_sizeof ("__ARM_SIZEOF_WCHAR_T",	\
+				    wchar_type_node);		\
 	if (TARGET_ARM_ARCH_PROFILE)				\
 	  builtin_define_with_int_value (			\
 	    "__ARM_ARCH_PROFILE", TARGET_ARM_ARCH_PROFILE);	\

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2014-08-12 15:01 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-12 15:01 [Patch ARM] Fix PR target/61413 - definition of __ARM_SIZEOF_WCHAR_T Ramana Radhakrishnan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).