------- Additional Comments From mark at codesourcery dot com 2004-10-22 16:26 ------- Subject: Re: [4.0 regression] mmintrin.h rejected by C++ frontend
reichelt at gcc dot gnu dot org wrote: >------- Additional Comments From reichelt at gcc dot gnu dot org 2004-10-22 08:00 >------- >Fixed on mainline. > >Mark, your patch http://gcc.gnu.org/ml/gcc-cvs/2004-10/msg01201.html >adds __extension__ to __gthread_active_p to solve the issue in comment #2. > >However, this was already fixed by Nathan's patch >http://gcc.gnu.org/ml/gcc-cvs/2004-10/msg01199.html >so the __extension__ shouldn't be necessary. > >Are you going to remove it again, or would you rather keep it? > Yes, Nathan and I stepped on each other there. I think we should keep the __extension__ just for clarity; it is an extension, and Jason, Nathan, and I have been contemplating issuing a diagnostic, but having it be a warning. It probably wouldn't trigger since this is a system header, but I think it's best to leave the marker there. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=18073