-
Notifications
You must be signed in to change notification settings - Fork 626
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
Enable libstdc++ v2 #146
Open
Sermus
wants to merge
20
commits into
pfalcon:master
Choose a base branch
from
Sermus:enable_libstdc++_v2
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Enable libstdc++ v2 #146
Changes from all commits
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
2b1466d
crosstool-NG was switched to xtensa-1.22.x. 1000-mforce-l32.patch was…
Sermus 15b1ddd
libnewlibport.a library added
Sermus e155241
libstdc++port library pulled in
Sermus e94334a
Added copying eagle.app.v6.new.2048.ld to toolchain lib directory. Ad…
Sermus e49011b
Build proceure for libstdc++irom added
Sermus 6eb43c1
esp_newlib_port changes pulled in
Sermus 0f6e9c4
New version of esp_stdcpp_port pulled in
Sermus c6c0de4
*irom.a:(.text .literal .text.* .literal.*) go to irom instead of *(.…
Sermus f12244f
Updated to the latest upstream
Sermus 85469d0
esp8266_sgi_stl_tests added as an example
Sermus fc46726
Further cleanup in esp_stdcpp_port and SGI STL tests
Sermus dc6db5b
Building libmirom.a is added to Makefile. .text and .literal sections…
Sermus 1706c91
add esp_stdcpp_port to clean target
jeremyd2019 661e4bd
Merge pull request #1 from jeremyd2019/enable_libstdc++_v2
Sermus 6a6596d
Update README.md
balazsracz 2d1bc6c
Merge pull request #2 from balazsracz/patch-1
Sermus fea6183
Merge back the latest upstream
Sermus 464d551
Update libnewlibport
Sermus 26910ae
Merge with upstream
Sermus f9765de
new crosstool
Sermus File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,45 @@ | ||
| This patch fixes vendor SDK header "c_types.h" for C99 compatibility. | ||
|
|
||
| --- ESP8266_NONOS_SDK_V2.0.0_16_07_19/include/c_types.h.orig 2016-07-18 07:16:07.000000000 -0700 | ||
| +++ ESP8266_NONOS_SDK_V2.0.0_16_07_19/include/c_types.h 2016-07-26 21:45:18.904657700 -0700 | ||
| @@ -6,17 +6,19 @@ | ||
| #ifndef _C_TYPES_H_ | ||
| #define _C_TYPES_H_ | ||
|
|
||
| -typedef unsigned char uint8_t; | ||
| +#include <stdint.h> | ||
| +#include <stdbool.h> | ||
| +//typedef unsigned char uint8_t; | ||
| typedef signed char sint8_t; | ||
| -typedef signed char int8_t; | ||
| -typedef unsigned short uint16_t; | ||
| +//typedef signed char int8_t; | ||
| +//typedef unsigned short uint16_t; | ||
| typedef signed short sint16_t; | ||
| -typedef signed short int16_t; | ||
| -typedef unsigned int uint32_t; | ||
| +//typedef signed short int16_t; | ||
| +//typedef unsigned int uint32_t; | ||
| typedef signed long sint32_t; | ||
| -typedef signed int int32_t; | ||
| +//typedef signed int int32_t; | ||
| typedef signed long long sint64_t; | ||
| -typedef unsigned long long uint64_t; | ||
| +//typedef unsigned long long uint64_t; | ||
| typedef unsigned long long u_int64_t; | ||
| typedef float real32_t; | ||
| typedef double real64_t; | ||
| @@ -82,10 +84,10 @@ | ||
| #define STORE_ATTR __attribute__((aligned(4))) | ||
|
|
||
| #ifndef __cplusplus | ||
| -typedef unsigned char bool; | ||
| +//typedef unsigned char bool; | ||
| #define BOOL bool | ||
| -#define true (1) | ||
| -#define false (0) | ||
| +//#define true (1) | ||
| +//#define false (0) | ||
| #define TRUE true | ||
| #define FALSE false | ||
|
|
Submodule crosstool-NG
updated
43 files
Submodule esp_newlib_port
added at
20d62a
Submodule esp_stdcpp_port
added at
77e48e
This file contains 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
Submodule esp8266_sgi_stl_tests
added at
f5464c
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since it has pretty much been established that this
objcopydoesn't do anything useful, might it not be more efficient to just usecpinstead?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tend to agree, but there is one argument against this approach. I tried to keep backward compatibility meaning the patch mustn't break solutions which worked before. If we change libstdc++irom just to a copy we either keep inconsistency between the way how libcirom is built and the way how libstdc++irom is built or we may break those solutions which used libcirom assuming that the functions inside are packed into .irom0.text.
So i'd rather keep it the way it's done now to keep backward compatibility and unified way of building *irom libs.