This is the mail archive of the libstdc++@gcc.gnu.org mailing list for the libstdc++ project.


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

Re: V3 PATCH: Fix PR3988



> > Well, a patch for config/os/gnu-linux/bits/os_defines.h or a solution
> > like  the 386 one that Mark did?

Please try to keep this cpu-specific stuff out of os-specific headers. 

> I think the CPU-specific way is better, in general.  I think that David
> asked me about doing that way, originally, and I chickened out; either
> way does get the job done.

Only, putting the cpu-stuff in config/cpu actually makes sense. This 
avoids confusion, and makes future maintenance enjoyable.

> If most of the systems on PowerPC do this the same way, it makes
> sense to use the CPU-specific way.  I'd prefer not to change existing
> configurations, though, if we can help it; there's no need to introduce
> risk on AIX, for example, if things already work correctly there.

Please do this correctly on the trunk, at least.

-benjamin


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