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

More reliable toolchain for ststm32@~14.1.0 ? #25846

Conversation

thinkyhead
Copy link
Member

One report of this toolchain being more reliable. Is it?

@thisiskeithb
Copy link
Member

Please define "more reliable".

Hopefully that report(er) did extensive testing since this affects a lot of users.

@thinkyhead thinkyhead force-pushed the bf2_reliable_stm_toolchain_PR branch from 397a347 to d81630e Compare May 22, 2023 01:52
@thinkyhead
Copy link
Member Author

Please define "more reliable".

In his case the older C++ compiler didn't do a very good job with -flto Link-Time Optimization, but the newer compiler did alright. We're already placing a lot of faith in the compilers to do the right thing, and we have hit issues before with certain optimizations and certain compilers. Presumably, newer compilers have fewer bugs.

@thinkyhead thinkyhead merged commit 317450a into MarlinFirmware:bugfix-2.1.x May 24, 2023
@thinkyhead thinkyhead deleted the bf2_reliable_stm_toolchain_PR branch May 24, 2023 07:53
ncksnydr added a commit to ncksnydr/Narungol-Marlin-Firmware that referenced this pull request May 24, 2023
… into feature/bugfix-2.1.x

* 'bugfix-2.1.x' of https://github.com/MarlinFirmware/Marlin:
  🩹 Fix ESP32 compile error (MarlinFirmware#25873)
  🧑‍💻 Updated toolchain for ststm32@~14.1.0 (MarlinFirmware#25846)
  🧑‍💻 Misc. ExtUI LCD cleanup (MarlinFirmware#25872)
  [cron] Bump distribution date (2023-05-24)
  ✅ Update checkout actions
  [cron] Bump distribution date (2023-05-23)
tspiva pushed a commit to tspiva/Marlin that referenced this pull request May 25, 2023
Andy-Big pushed a commit to Andy-Big/Marlin_FB_Reborn that referenced this pull request Jul 15, 2023
Andy-Big pushed a commit to Andy-Big/Marlin_FB_Reborn that referenced this pull request Jul 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants