sdcc not accepting code

Question

I have an issue with SDCC. My code (which I am attempting to port from another compiler) uses structs with flexible array members. However, when I try to compile the following code:

/** header of string list */
typedef struct {
    int nCount;
    int nMemUsed;
    int nMemAvail;
} STRLIST_HEADER;

/** string list entry data type */
typedef struct {
    int nLen;
    char str[];
} STRLIST_ENTRY;

/** string list data type */
typedef struct {
    STRLIST_HEADER header;
    STRLIST_ENTRY entry[];
} STRLIST;                      // By the way, this is the line the error refers to.

int main()
{
    return 0;
}

SDCC gives the following error:

$ sdcc -mz80 -S --std-c99 test.c
test.c:18: warning 186: invalid use of structure with flexible array member
test.c:18: error 200: field 'entry' has incomplete type

What gives? This code compiles just fine in gcc, not to mention the other z80 compiler I'm using.

EDIT: I found this SDCC bug which seems to be related. Could someone explain if it is and how?

score 3 · Accepted Answer · answered Jan 16 '13 at 06:52

SDCC is right there, and gcc-4.6.2 does not compile it "just fine" either. Well, if you ask it to adhere to the standard pedantically.

Compiling with -std=c99 -pedantic (or -std=c1x -pedantic), gcc emits

warning: invalid use of structure with flexible array member

and clang-3.0 behaves similarly, its warning is slightly more informative:

warning: 'STRLIST_ENTRY' may not be used as an array element due to flexible array member

The standard prohibits that in 6.7.2.1 (3):

A structure or union shall not contain a member with incomplete or function type (hence, a structure shall not contain an instance of itself, but may contain a pointer to an instance of itself), except that the last member of a structure with more than one named member may have incomplete array type; such a structure (and any union containing, possibly recursively, a member that is such a structure) shall not be a member of a structure or an element of an array.

(emphasis is mine)

gcc and clang allow having structs with flexible array members as members of structs or arrays as an extension. The standard prohibits that, so code using that is not portable, and every compiler is within its rights to reject the code.

The linked issue is not relevant, it is about not giving a warning if a struct with a flexible array member is instantiated as an automatic, which isn't allowed per the standard (but accepted by SDCC and others as an extension).

Excellent. The SDCC documentation is poor at best, and GCC's acceptance of the statement served to further confuse me. Thank you very much! — George Hilliard, Jan 16 '13 at 07:30

score 1 · Answer 2 · answered Jul 17 '13 at 07:58

Think about why this is an error.

STRLIST_ENTRY is of an unknown size because str[] is variable length.

STRLIST contains a variable length array of STRLIST_ENTRY.

In memory the array is a sequence of STRLIST_ENTRY laid out immediately after one another.

Since the elements of that are of an unknown size, how is it possible to know the offset an index points to?

char str[] should either be given a fixed size, or made a char* to a string outside of the array.

score 0 · Answer 3 · answered Aug 18 '21 at 14:42

I know I'm 8 years too late :), but perhaps it helps someone else. Here's the trick to use for data at the end of structures (i.e. when you have a header and last member is data of unknown size):

typedef struct {
    int nLen;
    char str[0];
} STRLIST_ENTRY;

This will let you address any char inside the string with an index i.e. str[10] and char *ptr=str will also give you the correct result.

sdcc not accepting code

3 Answers3

Linked