https://llvm.org/bugs/show_bug.cgi?id=25575

            Bug ID: 25575
           Summary: clang shoudn't ignore packed attribute for bit-field
                    of char type
           Product: clang
           Version: trunk
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P
         Component: Frontend
          Assignee: unassignedclangb...@nondot.org
          Reporter: dmitry.poluk...@gmail.com
                CC: llvm-bugs@lists.llvm.org
    Classification: Unclassified

Test case:
struct S1 {
  char a : 4;
  char b : 8 __attribute__((packed));
  char c : 4;
};

struct S2 {
  char a : 4;
  int b : 8 __attribute__((packed));
  char c : 4;
};

extern int a[sizeof(struct S1) == 2 ? 1 : -1];
extern int b[sizeof(struct S2) == 2 ? 1 : -1];

Clang ignores packed attribute and reports error:
test.c:3:29: warning: 'packed' attribute ignored for field of type 'char'
[-Wignored-attributes]
  char b : 8 __attribute__((packed));
                            ^
test.c:13:14: error: 'a' declared as an array with a negative size
extern int a[sizeof(struct S1) == 2 ? 1 : -1];
             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning and 1 error generated.

GCC 4.4+ only emits warning but compiles successfully:
test.c:5:1: note: offset of packed bit-field ‘b’ has changed in GCC 4.4
 };
 ^

According to GCC documentation the behavior was changes in 4.4:
Note: The 4.1, 4.2 and 4.3 series of GCC ignore the packed attribute on
bit-fields of type char. This has been fixed in GCC 4.4 but the change can lead
to differences in the structure layout. See the documentation of
-Wpacked-bitfield-compat for more information. 

So clang emulates old GCC behavior that doesn't have much sense and was fixed
in newer GCC versions. Input on why current behavior should be preserved or
changed is very appreciated.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs

Reply via email to