On 17/12/14 00:29, Joseph Myers wrote:
On Tue, 16 Dec 2014, Renlin Li wrote:

Hi all,

According to ACLE 2.0, the value of __ARM_SIZEOF_WCHAR_T should be defined in
terms of byte, which means it should be 2 or 4. This patch corrects the error
in arm backend.
What error?  builtin_define_type_sizeof does define a size in bytes (it
uses TYPE_SIZE_UNIT).  WCHAR_TYPE_SIZE should die, as all the relevant
information can be deduced from WCHAR_TYPE (and should be deduced via
MODIFIED_WCHAR_TYPE being used to set wchar_type_node, so that
-fshort-wchar is handled correctly).

Hi Joseph,

Yes, I have checked the code, and you are right. Thank you for explaining it. Please ignore this patch.

Regards,
Renlin Li

Reply via email to