Bug creation and email sending has been disabled, file new bugs at gcc.gnu.org/bugzilla
Bug 11 - ARM: runnable/test11.d Unsupported platform
Summary: ARM: runnable/test11.d Unsupported platform
Status: RESOLVED FIXED
Alias: None
Product: GDC
Classification: Unclassified
Component: gdc (show other bugs)
Version: development
Hardware: ARM Linux
: --- trivial
Assignee: Johannes Pfau
URL:
Depends on:
Blocks:
 
Reported: 2012-09-25 11:02 CEST by Johannes Pfau
Modified: 2012-09-26 08:32 CEST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Johannes Pfau 2012-09-25 11:02:28 CEST
This test is currently failing:
----
    byte* p;

    version(X86)
        assert(p.sizeof == 4);
    else version(X86_64)
        assert(p.sizeof == 8);
    else
        assert(false, "unknown platform");
----

we could of course just add a
----
    else version(ARM)
        assert(p.sizeof == 4);
----

but I wonder whether this should use D_LP64 instead? With armv8/AArch64 arm will support 64bit pointers. The question then is whether we introduce a new version for 64 bit version(ARM64), or whether version(ARM) could be 32 or 64 bit and we should additionally use version(D_LP64)?
Comment 1 Iain Buclaw 2012-09-25 16:35:55 CEST
Yep, it should be D_LP64.  Raise it upstream. Alexrp should agree with me. :^)
Comment 3 Johannes Pfau 2012-09-26 08:32:30 CEST
Fixed upstream in dmd 2.061
https://github.com/D-Programming-Language/dmd/commit/375855698cd4de3c74f9840941daf37b9ae36dcf