Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build system: Changes for xc32 compiler #1264

Merged
merged 1 commit into from
Dec 31, 2021

Conversation

kasjer
Copy link
Collaborator

@kasjer kasjer commented Dec 27, 2021

Describe the PR
Three changes are needed to accommodate xc32 compiler build:

  • optimized build flag other than -Os
    added CFLAGS_OPTIMIZED that defaults to -Os but can be overridden in boards
  • build without -lnosys
    added LIBS_GCC with default libraries that can be changed in boards
  • build without LD_FILE specification
    if LD_FILE is empty -Wl,-T options is not added to LDFLAGS

Additional context
xc32 compiler for Microchip mips chips in free edition have some functional limitations like
lack of -Os and -flto.
It also provides linker script automatically so it does not have to be specified on command line.
It does not have nosys library that is not needed for bulild.

Three changes are needed to accommodate xc32 compiler build:
- optimized build flag other than -Os
   added CFLAGS_OPTIMIZED that defaults to -Os but can be overridden in boards
- build without -lnosys
   added LIBS_GCC with default libraries that can be changed in boards
- build without LD_FILE specification
   if LD_FILE is empty -Wl,-T options is not added to LDFLAGS
Copy link
Owner

@hathach hathach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

look great, thank you very much for pr

@hathach hathach merged commit 868948f into hathach:master Dec 31, 2021
@kasjer kasjer deleted the kasjer/make-files-for-xc32 branch December 31, 2021 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants