FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fixes for arm-none-eabi-g++ by matgla · Pull Request #1045 · hsutter/cppfront · GitHub

Fixes for arm-none-eabi-g++ - #1045

Merged
hsutter merged 3 commits into
hsutter:mainfrom
matgla:fixNoRtti
May 11, 2024
Merged

hsutter merged 3 commits into
hsutter:mainfrom
matgla:fixNoRtti

Conversation

matgla commented Mar 29, 2024

Copy link
Copy Markdown
Contributor

Typeid cannot be used when -fno-rtti is used.
Fixed compilation of generated sources on arm-none-eabi-g++ 13.1.

Fixed compilation of generated sources on arm-none-eabi-g++ 13.1.

matgla commented Mar 29, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Hi,
I just wanted to raise issue that fix is necessary for -fno-rtti + -fno-exception builds.
Usage of PRETTY_FUNCTION and MSVC FUNCSIG is a little bit hacky I think, so maybe it's better to print just exception without type.

arm-none-eabi-g++ also fails on std::string to const char* conversion.

Gotbolt link with example usage compile time type_name : https://godbolt.org/z/75fnc5Krq

Real use example:
09:47:22.2417 [INFO] uart0: [host: 81.23ms (+1.83ms)|virt: 0.2ms (+0.2ms)] Type safety violation: Throw exception (-fno-exception) with type "std::error_condition&&" and the message "'fopen' attempt failed"

hsutter commented May 8, 2024

Copy link
Copy Markdown
Owner

Thanks! That's quite cute.

It looks like this may be your first contribution to cppfront. I've sent the Contributor License Agreement (CLA) to your email, and once it's signed I can look in more detail at your pull request. Thanks again for your contribution.

hsutter commented May 11, 2024

Copy link
Copy Markdown
Owner

Looks, good, thanks!

hsutter merged commit bfc1f45 into hsutter:main May 11, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL