Re: [patch,avr,installed] ad PR65296: Adjust specs to new avr-libc layout as of #44574
2015-04-17 17:02 GMT+03:00 Georg-Johann Lay a...@gjlay.de:
...I went ahead and installed as
http://gcc.gnu.org/r222179
It will be backported to 5.2 as soon as 5.1 is open for patches again
(assuming RM won't approve this one for 5.1).
IMHO AVR port is not locked for patches.
It's not a primary target.
As far as I can tell, all works fine now, even with install-paths containing
spaces and LTO.
Johann
2015-04-17 Sivanupandi Pitchumani pitchumani.sivanupa...@atmel.com
PR target/65296
* config/avr/gen-avr-mmcu-specs.c (*avrlibc_startfile): Adjust
to new AVR-LibC file layout (bug #44574).
(*avrlibc_devicelib): Same.
* config/avr/avr-mcus.def: Adjust comments.
* config/avr/avr.opt (nodevicelib): Adjust help.
Index: config/avr/gen-avr-mmcu-specs.c
===
--- config/avr/gen-avr-mmcu-specs.c (revision 222178)
+++ config/avr/gen-avr-mmcu-specs.c (revision 222179)
@@ -171,11 +171,11 @@ bool is_arch = NULL == mcu-macro;
if (is_device)
{
fprintf (f, *avrlibc_startfile:\n);
- fprintf (f, \tdev/%s/crt1.o%%s, mcu-name);
+ fprintf (f, \tcrt%s.o%%s, mcu-name);
fprintf (f, \n\n);
fprintf (f, *avrlibc_devicelib:\n);
- fprintf (f, \t%%{!nodevicelib:dev/%s/libdev.a%%s}, mcu-name);
+ fprintf (f, \t%%{!nodevicelib:-l%s}, mcu-name);
fprintf (f, \n\n);
}
#endif // WITH_AVRLIBC
Index: config/avr/avr-mcus.def
===
--- config/avr/avr-mcus.def (revision 222178)
+++ config/avr/avr-mcus.def (revision 222179)
@@ -44,8 +44,8 @@ Before including this file, define a mac
used by DRIVER_SELF_SPECS and gen-avr-mmcu-specs.c for
- the name of the device specific specs file
in -specs=device-specs/spec-NAME
- - the name of the startup file dev/NAME/crt1.o
- - the name of the device library dev/NAME/libdev.a
+ - the name of the startup file crtNAME.o
+ - to link the device library by means of -lNAME
ARCH Specifies the multilib variant together with
AVR_SHORT_SP
Index: config/avr/avr.opt
===
--- config/avr/avr.opt (revision 222178)
+++ config/avr/avr.opt (revision 222179)
@@ -97,4 +97,4 @@ Allow to use truncation instead of round
nodevicelib
Driver Target Report RejectNegative
-Do not link against the device-specific library libdev.a
+Do not link against the device-specific library libMCU.a
[patch,avr,installed] ad PR65296: Adjust specs to new avr-libc layout as of #44574
...I went ahead and installed as
http://gcc.gnu.org/r222179
It will be backported to 5.2 as soon as 5.1 is open for patches again (assuming
RM won't approve this one for 5.1).
As far as I can tell, all works fine now, even with install-paths containing
spaces and LTO.
Johann
2015-04-17 Sivanupandi Pitchumani pitchumani.sivanupa...@atmel.com
PR target/65296
* config/avr/gen-avr-mmcu-specs.c (*avrlibc_startfile): Adjust
to new AVR-LibC file layout (bug #44574).
(*avrlibc_devicelib): Same.
* config/avr/avr-mcus.def: Adjust comments.
* config/avr/avr.opt (nodevicelib): Adjust help.
Index: config/avr/gen-avr-mmcu-specs.c
===
--- config/avr/gen-avr-mmcu-specs.c (revision 222178)
+++ config/avr/gen-avr-mmcu-specs.c (revision 222179)
@@ -171,11 +171,11 @@ bool is_arch = NULL == mcu-macro;
if (is_device)
{
fprintf (f, *avrlibc_startfile:\n);
- fprintf (f, \tdev/%s/crt1.o%%s, mcu-name);
+ fprintf (f, \tcrt%s.o%%s, mcu-name);
fprintf (f, \n\n);
fprintf (f, *avrlibc_devicelib:\n);
- fprintf (f, \t%%{!nodevicelib:dev/%s/libdev.a%%s}, mcu-name);
+ fprintf (f, \t%%{!nodevicelib:-l%s}, mcu-name);
fprintf (f, \n\n);
}
#endif // WITH_AVRLIBC
Index: config/avr/avr-mcus.def
===
--- config/avr/avr-mcus.def (revision 222178)
+++ config/avr/avr-mcus.def (revision 222179)
@@ -44,8 +44,8 @@ Before including this file, define a mac
used by DRIVER_SELF_SPECS and gen-avr-mmcu-specs.c for
- the name of the device specific specs file
in -specs=device-specs/spec-NAME
- - the name of the startup file dev/NAME/crt1.o
- - the name of the device library dev/NAME/libdev.a
+ - the name of the startup file crtNAME.o
+ - to link the device library by means of -lNAME
ARCH Specifies the multilib variant together with AVR_SHORT_SP
Index: config/avr/avr.opt
===
--- config/avr/avr.opt (revision 222178)
+++ config/avr/avr.opt (revision 222179)
@@ -97,4 +97,4 @@ Allow to use truncation instead of round
nodevicelib
Driver Target Report RejectNegative
-Do not link against the device-specific library libdev.a
+Do not link against the device-specific library libMCU.a
Re: [patch,avr,installed] ad PR65296: Adjust specs to new avr-libc layout as of #44574
Am 04/17/2015 um 04:43 PM schrieb Denis Chertykov: 2015-04-17 17:02 GMT+03:00 Georg-Johann Lay a...@gjlay.de: ...I went ahead and installed as http://gcc.gnu.org/r222179 It will be backported to 5.2 as soon as 5.1 is open for patches again (assuming RM won't approve this one for 5.1). IMHO AVR port is not locked for patches. It's not a primary target. hmm, the usual text is that the complete branch is frozen: the branches/gcc-5-branch has been created last night and GCC 5.1-rc1 built and announced. The branch is now frozen for blocking regressions and documentation fixes only, all changes to the branch require a RM approval now. https://gcc.gnu.org/ml/gcc/2015-04/msg00145.html Johann
Re: [patch,avr,installed] ad PR65296: Adjust specs to new avr-libc layout as of #44574
2015-04-17 18:32 GMT+03:00 Georg-Johann Lay a...@gjlay.de: Am 04/17/2015 um 04:43 PM schrieb Denis Chertykov: 2015-04-17 17:02 GMT+03:00 Georg-Johann Lay a...@gjlay.de: ...I went ahead and installed as http://gcc.gnu.org/r222179 It will be backported to 5.2 as soon as 5.1 is open for patches again (assuming RM won't approve this one for 5.1). IMHO AVR port is not locked for patches. It's not a primary target. hmm, the usual text is that the complete branch is frozen: I asked this question a few years ago. The answe was that I can change the port in any time. Maybe something has changed.
