Andrew Stubbs ams@codesourcery.com wrote:
CS has this patch in SG++: http://gcc.gnu.org/ml/gcc-patches/2008-12/msg00199.html
This patch improves code size in a useful, target independent way, but was not committed upstream. It's not clear why. Since the patch does not belong to CodeSourcery, we can't upstream it ourselves either.
Steven asked: "What do you folks think of this patch?" and the only answer was by Richi: "I think it's reasonable."
So I'm not sure why it didn't go in. Maybe we should just ping Steven?
Is that patch a suitable candidate for Linaro GCC?
It is not upstreamable due to copyright issues, but we have a policy that we can keep such patches, if we wish.
What copyright issues? Steven ought to have an assignment in place ...
The principle of not letting Linaro and SG++ diverge too far also suggests keeping it.
Any thoughts? If nobody objects soon I shall merge it in.
Since there don't appear to be any fundamental objections to inclusion of the patch into mainline, and since the patch doesn't introduce any user-visible feature (like syntax extensions or command line options) that could cause build breaks for users if it were to go away after all, I would say this patch is fine for us ...
Mit freundlichen Gruessen / Best Regards
Ulrich Weigand
-- Dr. Ulrich Weigand | Phone: +49-7031/16-3727 STSM, GNU compiler and toolchain for Linux on System z and Cell/B.E. IBM Deutschland Research & Development GmbH Vorsitzender des Aufsichtsrats: Martin Jetter | Geschäftsführung: Dirk Wittkopp Sitz der Gesellschaft: Böblingen | Registergericht: Amtsgericht Stuttgart, HRB 243294