This is the mail archive of the gcc-bugs@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug sanitizer/59009] libsanitizer merge from upstream r191666 breaks bootstrap on powerpc64-linux


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

--- Comment #31 from dave.anglin at bell dot net ---
On 12/10/2013 11:49 AM, jakub at gcc dot gnu.org wrote:
> For that a patch has been posted, just not reviewed:
> http://gcc.gnu.org/ml/gcc-patches/2013-12/msg00963.html
Excellent.
>
> As for hppa, the change to enable it isn't in upstream GCC, so you as a
> maintainer need to make it working first and if changes are needed for
> compiler-rt maintained code, make sure it is accepted there first too.
I'm fully aware that the change to enable isn't upstream.  I'm still 
debating in my mind whether
this should be done or not.  At the moment, there are no additional 
changes to the compiler-rt
code as far as I know. However, things seem fragile and possibly this 
library is unsuitable for
an architecture that is only supported by Gentoo and Debian unstable 
Linux ports.

I'm not sure what the issue is with the hppa kernel typedef for time_t.  
It seems defined by the
generic linux type definitions.  If this needs to be changed for 
libsanitizer, I would like to
understand why.  As far as I can tell, hppa isn't unique in using time_t 
in uapi/asm/stat.h
although the majority of ports only use base types.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]