Sergey A. Borshch schrieb:
Chris Liechti wrote:
It would be great when some of you could take the time and test this installer and give feedback. In a test project I noticed code changes, and it got smaller, but i did not yet fully checked that everything still runs as expected.
1) text files (sources, linker scripts, readme) need line ending conversion before generating windows package. Readme file, which appears in windows notepad just after installation looks curiously.

hm. yeah i built it with LF line endings. need to check if i can build it with DOS endings again. arghhh such an endless story.. building software on windoze..

2)make.exe requires cygintl-8.dll, which is not included to installer.

thanks, fixed

3)constructors/destructors jf staitc not called from startup code (as in previous version, needs libmspgcc.S fixes).

ok.. some knowledgeable person ready to submit patches?

4) --fdata-sections generates internal compiler error. I think it must generate message that --fdata-sections not supported.

or the function should work, maybe it could be useful for the
devices >64kB Flash? (putting data in sections above 64k?)

5) uninstall does not delete root folder (c:/mspgcc).

that is because it is not empty. if it is a file that was installed it's a bug, but it's common that the examples folder is left over as the compilation generates files when a user tries them out.

6)--ffunction-sections with --gc-sections causes all code to be removed from output file. Needs corrections of linker scripts:
   .vectors  :
   {
      PROVIDE (__vectors_start = .) ;
     *(.vectors*)
     KEEP(*(.vectors*))

also, KEEP() must be added to all .initX, .finiX, .ctors and .dtors sections.

ok, looks like an easy thing, there are just so many files... ;-)

would there be a similar keyword to ensure that an empty bss section is also included in the binaries? it seems that GDB can not load files where no global variables are declared.

7) I think liker scripts can be simplified by repalcing
   .data   : AT (ADDR (.text) + SIZEOF (.text))
   {
      .........
   }  > data

with
   .data
   {
      .........
   }  > data AT > text


I think I can make a patch to fix linker scripts if required.

that would be great. while i think i understand your changes, i'm not having much practice working with these files.

chris

Reply via email to