This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch,wwwdocs] Add gcc-5 caveats for avr.
- From: Gerald Pfeifer <gerald at pfeifer dot com>
- To: Georg-Johann Lay <avr at gjlay dot de>
- Cc: gcc-patches at gcc dot gnu dot org, Denis Chertykov <chertykov at gmail dot com>
- Date: Mon, 20 Apr 2015 21:02:55 +0200 (CEST)
- Subject: Re: [patch,wwwdocs] Add gcc-5 caveats for avr.
- Authentication-results: sourceware.org; auth=none
- References: <5534D76F dot 2060503 at gjlay dot de>
Hi Johann,
On Mon, 20 Apr 2015, Georg-Johann Lay wrote:
Okay to install?
+ <li>The AVR port uses a new scheme to describe supported devices:
+ For each supported device the compiler provides a device-specific
+ <a href="http://gcc.gnu.org/onlinedocs/gcc/Spec-Files.html">spec file</a>.
+ If the compiler is used together with AVR-LibC, this requires at
+ least GCC 5.2 and a version of AVR-LibC which implements
+ <a href="http://savannah.nongnu.org/bugs/?44574"</a>#44574</a>.</li>
Can you please make the two links https-links? (Especially the
one to gcc.gnu.org actually redirects.)
Just using "#44574" for a reference, may that be a little confusing,
or is it sufficiently clear to AVR users?
+ <li>A new command option <code>-nodevicelib</code> has been added.
"command-line option"
+ If this option is turned on the compiler won't link against AVR-LibC's
+ device-specific library <code>lib<i>device</i>.a</code> by omitting
+ <code>-l<i>device</i></code> from the linker's command line.
How about making this "...-nodevicelib</code> prevents the compiler
from linking against...."?
+ If the compiler had not been
+ <a href="http://gcc.gnu.org/install/configure.html">configured</a>
+ to be used with AVR-LibC, the compiler will not link against that
+ library and the option has no effect.</li>
"was not" (or "is") instead of "had not", and can you please use
https here as well?
Though, really, could this be just simplified to "If the compiler is
not configured for use with AVR-LibC to begin with, this option has
no effect"?
Your patch is fine with the above changes or considering them and
deciding not go for one or the other.
Gerald