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

According to Frank so LTO works #358

Open
wants to merge 1 commit into
base: master
from

Conversation

@Defragster
Copy link
Contributor

commented Feb 21, 2019

https://forum.pjrc.com/threads/54711-Teensy-4-0-First-Beta-Test?p=198210&viewfull=1#post198210

Handler survives LTO pruning
Remove Printing when PRINT_DEBUG_STUFF #ifdef off

@Defragster

This comment has been minimized.

Copy link
Contributor Author

commented Mar 30, 2019

Paul, LTO may not be working - at least executing - but without this added change it won't compile:

Add line 266::
void HardFault_HandlerC(unsigned int *hardfault_args) attribute((used));

@Defragster

This comment has been minimized.

Copy link
Contributor Author

commented Mar 30, 2019

Cross ref - LTO compile/upload on T4 required 15s reset as noted here: https://forum.pjrc.com/threads/54711-Teensy-4-0-First-Beta-Test?p=202037&viewfull=1#post202037

@Defragster

This comment has been minimized.

Copy link
Contributor Author

commented Jul 16, 2019

Paul with the ASM code embedding the call to the Fault_Handler it is missed when 'weak' by LTO parsing. This (used) decoration with an added 'prototype' suggested by FrankB prevents that loss.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.