-
Notifications
You must be signed in to change notification settings - Fork 26
Preprocessor for ULP/RTC macros #43
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
79db90f
add units test for the .set directive
wnienhaus 84d734d
add support for left aligned assembler directives (e.g. .set)
wnienhaus ec81ecc
fix a crash bug where BSS size calculation was attempted on the value…
wnienhaus c184924
raise error when attempting to store values in .bss section
wnienhaus 25d34b0
fix reference to non-existing variable
wnienhaus 76a81ac
fix typo in comment of instruction definition
wnienhaus 56f4530
add support for the .global directive. only symbols flagged as global…
wnienhaus 9907b10
let SymbolTable.export() optionally export non-global symbols too
wnienhaus 27ab850
support ULP opcodes in upper case
wnienhaus 54b117e
add a compatibility test for the recent fixes and improvements
wnienhaus feb42dc
add support for evaluating expressions
wnienhaus 87507c9
add a compatibility test for evaluating expressions
wnienhaus 99352a3
docs: add that expressions are now supported
wnienhaus d76fd26
add preprocessor that can replace simple #define values in code
wnienhaus 4dded94
allow assembler to skip comment removal to avoid removing comments twice
wnienhaus 219f939
fix evaluation of expressions during first assembler pass
wnienhaus 5c3eeb8
remove no-longer-needed pass dependent code from SymbolTable
wnienhaus 3e8c0d5
add support for macros such as WRITE_RTC_REG
wnienhaus ac1de99
add simple include file processing
wnienhaus 8d88fd1
add support for using a btree database (DefinesDB) to store defines f…
wnienhaus 46f1442
add special handling for the BIT macro used in the esp-idf framework
wnienhaus 2f6ee78
add include processor tool for populating a defines.db from include f…
wnienhaus 69ae946
add compatibility tests using good example code off the net
wnienhaus 4f90f76
add documentation for the preprocessor
wnienhaus d44384f
fix use of treg field in i_move instruction to match binutils-esp32 o…
wnienhaus 254adf9
allow specifying the address for reg_rd and reg_wr in 32-bit words
wnienhaus c3bd101
support .int data type
wnienhaus 2a0a39a
refactor: small improvements based on PR comments.
wnienhaus 47d5e8a
Updated LICENSE file and added AUTHORS file
wnienhaus 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 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,44 @@ | ||
| .set adc_channel, 6 | ||
|
|
||
| .set adc_oversampling_factor_log, 2 | ||
| .set adc_oversampling_factor, (1 << adc_oversampling_factor_log) | ||
|
|
||
| .data | ||
|
|
||
| result: | ||
| .long 0 | ||
|
|
||
| .text | ||
| .global entry | ||
| entry: | ||
| move r0, 0 | ||
| stage_rst | ||
|
|
||
| measure: | ||
| adc r1, 0, adc_channel + 1 | ||
| add r0, r0, r1 | ||
|
|
||
| stage_inc 1 | ||
| jumps measure, adc_oversampling_factor, lt | ||
|
|
||
| rsh r0, r0, adc_oversampling_factor_log | ||
|
|
||
| move r3, result | ||
| st r0, r3, 0 | ||
|
|
||
| #test that expressions evaluate correctly for all supported operators | ||
| move r3, 1+2 | ||
| move r3, 3-5 | ||
| move r3, -5 | ||
| move r3, 2*3 | ||
| move r3, 4/2 | ||
| move r3, 4 % 3 | ||
| move r3, 0xff << 2 | ||
| move r3, 0xff >> 1 | ||
| move r3, (0xabcdef | 0xff) & 0xff | ||
| move r3, 0x1234 & ~2 | ||
| move r3, 42|4&0xf # 46 (4&0xf is evaluated first) | ||
| move r3, (42|4)&0xf # 14 (42|4 is evaluated first) | ||
|
|
||
| exit: | ||
| halt | ||
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.
this is a bit strange mix of a real-world code example and some lines of assembler testing.
maybe separate these lines at least?
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.
Did you mean into a separate file, or maybe I can move them below the
exit: haltline and add a comment that they are all "artificial" examples?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.
whatever you like better.