This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: PATCH: Move sysdeps/x86_64/Implies to sysdeps/x86_64/64
- From: Roland McGrath <roland at hack dot frob dot com>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: GNU C Library <libc-alpha at sourceware dot org>
- Date: Wed, 9 May 2012 15:54:20 -0700 (PDT)
- Subject: Re: PATCH: Move sysdeps/x86_64/Implies to sysdeps/x86_64/64
- References: <20120322175315.GA9938@intel.com><20120322195022.F2EF32C07C@topped-with-meat.com><CAMe9rOriXXrVWVLLuES8OUAXvbcVgcJEAYAgJjHWr3YCB2JmaQ@mail.gmail.com><CAMe9rOru6EXRsoqaBKS0a9fGqwey+52DCak0MwGPM1bzWf29NQ@mail.gmail.com><20120322204013.868A12C086@topped-with-meat.com><CAMe9rOq87mKXdoMOGH=cxiq8C6qhK7bTi5bAB9jQk_Eg7x72Vw@mail.gmail.com><20120322210647.330402C07E@topped-with-meat.com><CAMe9rOpiAWE8U2XFxgavJBi9FUomTZh3XEwcdYN59qNBR8OM7g@mail.gmail.com><20120322211655.C5E992C07E@topped-with-meat.com><CAMe9rOp7AVVWn_Bo6sKC7yRuCaRxiqUXGpLCm9wqvg2aSjHx-Q@mail.gmail.com><20120322220155.1AC112C07E@topped-with-meat.com><CAMe9rOqj3J=fw984tRMG-FTgxgxbKbjRZJzrnoaOfqzAi815+g@mail.gmail.com><20120322222523.5F52A2C08D@topped-with-meat.com><CAMe9rOqqKB0g_0xoueMWX-902eOeFqpw3LmO7Ej6p=xERjWtCQ@mail.gmail.com><20120406210447.642DE2C0C4@topped-with-meat.com><CAMe9rOo1gPRVRqhTwYjwdhKWyTgn2-aJMsLnHw7ZYZjPDnGiJw@mail.gmail.com><Pine.LNX.4.64.1204071511060.12109@digraph.polyomino.org.uk><CAMe9rOp_F85X-XJ4Jp-_AU1D_cDdZwEdMNMfKDsk4WgmtSWByg@mail.gmail.com>
The new branch roland/implies-before-after has a configure change to allow
both Implies-before and Implies-after files with what I think were the
semantics you wanted them to have. (It's only trivially different from the
patch I previously posted for just Implies-after.)
I'm certainly not happy with that as a solution, but I don't think we can
reasonably expect to manage anything requiring more deep thinking in the
time we want to get x32 merged in. (It will be all I can do just to do an
adequate review on all your patches in the next week or three.)
If that is sufficient for you to get things working adequately for x32,
then I think we should go with this change for now, even though it's ugly
and only confuses the situation in general terms. (But nobody else should
start using Implies-{before,after} files.)
After 2.16, we can look into better and more sustainable solutions such as
splitting up the wordsize-NN directories into ones with finer-grained and
more clear meanings and/or more substantially revamping the sysdeps
directory selection mechanism to be more comprehensible.
Thanks,
Roland