Fork us on GitHub Follow us on Google+ Follow us on Facebook Follow us on Twitter

Opened 18 months ago

Closed 4 months ago

#683 closed defect (fixed)

Clang build is broken on some architectures

Reported by: Jiri Svoboda Owned by: Jiří Zárevúcky
Priority: minor Milestone: 0.7.1
Component: helenos/unspecified Version: mainline
Keywords: Cc:
Blocker for: Depends on:
See also:

Description

As of changeset:mainline,2660, trying to build (amd64 profile) with Clang compiler, the build fails in autotool.py with:

Checking for genisoimage ... ok
Checking compiler properties ... failed
probe.c:36:2: error: unexpected token in argument list
        DECLARE_BUILTIN_TYPE("size", __SIZE_TYPE__);
        ^
probe.c:12:2: note: expanded from macro 'DECLARE_BUILTIN_TYPE'
        AUTOTOOL_DECLARE("builtin_size", "", tag, STRING(type), "", "", sizeof(type)); \
        ^
probe.c:3:3: note: expanded from macro 'AUTOTOOL_DECLARE'
                "AUTOTOOL_DECLARE\t" category "\t" subcategory "\t" tag "\t" name "\t" strc "\t" conc "\t%[val]\n" \
                ^

Change History (6)

comment:1 Changed 15 months ago by Jiří Zárevúcky

Owner: set to Jiří Zárevúcky
Status: newaccepted

comment:2 Changed 12 months ago by Jakub Jermář

Can this issue be closed now that HelenOS can be built by clang on amd64?

comment:3 Changed 12 months ago by Jiří Zárevúcky

Milestone: 0.7.10.7.2
Priority: majorminor
Summary: Clang build brokenClang build is broken on some architectures

It's still broken on some architectures. IMO this should be kept open until it works everywhere except IA64. Also, clang-based build should be tested by CI.

I have reduced the priority, though, since you can always just use GCC.

comment:4 Changed 12 months ago by Jiri Svoboda

Clang has ever only worked with amd64. If it works there again, I consider my original request addressed.

comment:5 Changed 12 months ago by Jiří Zárevúcky

In that case, we can either change this to an enhancement, or close and I'll open a separate enhancement issue for other architectures. Either way works for me.

comment:6 Changed 4 months ago by Jakub Jermář

Milestone: 0.7.20.7.1
Resolution: fixed
Status: acceptedclosed

I am closing this to motivate the prospective opening of finer-grained tickets.

Note: See TracTickets for help on using tickets.