This is the mail archive of the gcc-patches@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]

Re: patch for merging graphite branch (before tuplification)


Joseph S. Myers wrote:

GCC 4.4 -fwhatever should generate the same results everywhere. I think reproducibility of results across hosts, for the same compiler sources, is a critical matter. This means release branches must require a particular version, and check the version when the compiler executes if linking against a Cloog shared library is supported.

I agree with this statement of principle.


There is a logical extreme of this that I think unreasonable; for example, if we depend on the C library to do something, and it does it wrong on some host, then we may lose -- and it's not practical to include the C library as part of the compiler sources. But, people should know how to get reference versions of core computational libraries, and I know you're not arguing for the logical extreme.

I'm not sure, however, that we need to try too hard to prevent people using other versions. We just need to say "the version we used is X; if you want to try some other version, your mileage may vary." We need to agree on X (including what version is to be used on branches vs. mainline) so that if you report a bug in my change and I go to fix it I don't get confused because you and I are using different versions. A warning seems appropriate, but an error seems strong.

In the past, I've argued against importing third-party libraries into our tree, and I still feel that way. I've got no objection to canonical versions available from the gcc.gnu.org FTP site, but I'd rather not have them be part of every GCC checkout. I don't feel too strongly about this, and we compromised on GMP, but that's still my overall sentiment.

--
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713


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