public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c++/40527]  New: #pragma pack([push,] n) should be coded in the signature
@ 2009-06-23 12:39 gerolf dot wendland at nsn dot com
  2009-08-25 15:33 ` [Bug c++/40527] " bangerth at gmail dot com
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: gerolf dot wendland at nsn dot com @ 2009-06-23 12:39 UTC (permalink / raw)
  To: gcc-bugs

I do regard the alignment settings for structs, unions, 
classes as important as the signature of functions.

compilation of structs in the scope of different 
#pragma pack([push], n) 
produces errors that are hard to understand.

The linker could produce errors or warnings when
combining compilation units with symbols that differ
only in their memory alignment.


The compiler's warning about impaired 
#pragma pack(push,n)
#pragma pack(pop)

doesn't help in such cases.


-- 
           Summary: #pragma pack([push,] n) should be coded in the signature
           Product: gcc
           Version: 4.3.1
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: P3
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: gerolf dot wendland at nsn dot com


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40527


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Bug c++/40527] #pragma pack([push,] n) should be coded in the signature
  2009-06-23 12:39 [Bug c++/40527] New: #pragma pack([push,] n) should be coded in the signature gerolf dot wendland at nsn dot com
@ 2009-08-25 15:33 ` bangerth at gmail dot com
  2009-08-26  6:55 ` gerolf dot wendland at nsn dot com
  2009-11-10 18:53 ` jason at gcc dot gnu dot org
  2 siblings, 0 replies; 4+ messages in thread
From: bangerth at gmail dot com @ 2009-08-25 15:33 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #1 from bangerth at gmail dot com  2009-08-25 15:33 -------
That would require that the ABI specifies such a mangling. I'm not sure
anyone wants to go that route.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40527


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Bug c++/40527] #pragma pack([push,] n) should be coded in the signature
  2009-06-23 12:39 [Bug c++/40527] New: #pragma pack([push,] n) should be coded in the signature gerolf dot wendland at nsn dot com
  2009-08-25 15:33 ` [Bug c++/40527] " bangerth at gmail dot com
@ 2009-08-26  6:55 ` gerolf dot wendland at nsn dot com
  2009-11-10 18:53 ` jason at gcc dot gnu dot org
  2 siblings, 0 replies; 4+ messages in thread
From: gerolf dot wendland at nsn dot com @ 2009-08-26  6:55 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #2 from gerolf dot wendland at nsn dot com  2009-08-26 06:55 -------
(In reply to comment #1)
> That would require that the ABI specifies such a mangling. I'm not sure
> anyone wants to go that route.

This is exactly what I had in mind. Slightly extended names for
structs/classes/unions would eventually lead to different mangled
names of functions.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40527


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Bug c++/40527] #pragma pack([push,] n) should be coded in the signature
  2009-06-23 12:39 [Bug c++/40527] New: #pragma pack([push,] n) should be coded in the signature gerolf dot wendland at nsn dot com
  2009-08-25 15:33 ` [Bug c++/40527] " bangerth at gmail dot com
  2009-08-26  6:55 ` gerolf dot wendland at nsn dot com
@ 2009-11-10 18:53 ` jason at gcc dot gnu dot org
  2 siblings, 0 replies; 4+ messages in thread
From: jason at gcc dot gnu dot org @ 2009-11-10 18:53 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #3 from jason at gcc dot gnu dot org  2009-11-10 18:53 -------
This falls in the category of diagnosing ODR violations, which we don't really
try to do currently.  It would be possible to do some ODR checking based on the
debug info that we already emit; that seems a more promising route than
breaking the ABI for this sort of thing.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40527


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2009-11-10 18:53 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-06-23 12:39 [Bug c++/40527] New: #pragma pack([push,] n) should be coded in the signature gerolf dot wendland at nsn dot com
2009-08-25 15:33 ` [Bug c++/40527] " bangerth at gmail dot com
2009-08-26  6:55 ` gerolf dot wendland at nsn dot com
2009-11-10 18:53 ` jason at gcc dot gnu dot org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).