Re: [PATCH] Fix PR66805 - #pragma pack affecting gcov_info_type layout

2015-07-08 Thread Alexander Monakov
The same bug was earlier reported as PR gcov-profile/43341: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43341 Alexander

Re: [PATCH] Fix PR66805 - #pragma pack affecting gcov_info_type layout

2015-07-08 Thread Jakub Jelinek
On Wed, Jul 08, 2015 at 01:58:38PM +0200, Richard Biener wrote: The following fixes #pragma pack effect leaking to all types built from the middle-end (so possibly even vector types built by the vectorizer?). The PR in question is about gcov_info_type where layout is affected and

Re: [PATCH] Fix PR66805 - #pragma pack affecting gcov_info_type layout

2015-07-08 Thread Jakub Jelinek
On Wed, Jul 08, 2015 at 02:40:33PM +0200, Richard Biener wrote: toplev.c already sets maximum_field_alignment directly, and to a different value: maximum_field_alignment = initial_max_fld_align * BITS_PER_UNIT; so I'm not sure you need a new function. And, shouldn't you reset to

[PATCH] Fix PR66805 - #pragma pack affecting gcov_info_type layout

2015-07-08 Thread Richard Biener
The following fixes #pragma pack effect leaking to all types built from the middle-end (so possibly even vector types built by the vectorizer?). The PR in question is about gcov_info_type where layout is affected and inconsistency between that and the libgcov.a copy causes libgcov to crash. As

Re: [PATCH] Fix PR66805 - #pragma pack affecting gcov_info_type layout

2015-07-08 Thread Richard Biener
On Wed, 8 Jul 2015, Jakub Jelinek wrote: On Wed, Jul 08, 2015 at 01:58:38PM +0200, Richard Biener wrote: The following fixes #pragma pack effect leaking to all types built from the middle-end (so possibly even vector types built by the vectorizer?). The PR in question is about

Re: [PATCH] Fix PR66805 - #pragma pack affecting gcov_info_type layout

2015-07-08 Thread Richard Biener
On Wed, 8 Jul 2015, Richard Biener wrote: The following fixes #pragma pack effect leaking to all types built from the middle-end (so possibly even vector types built by the vectorizer?). The PR in question is about gcov_info_type where layout is affected and inconsistency between that and