-
Notifications
You must be signed in to change notification settings - Fork 811
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
Cortex-M/Thumb2 ASM: fix label #7753
Conversation
0211646
to
a18ea90
Compare
@SparkiDev I'm using wolfSSL with IAR v9.50.2 and haven't seen issues with |
a18ea90
to
19c6b28
Compare
Hi @josepho0918, I've modified the code to check the version of IAR. Thanks, |
Hi @josepho0918 Could you please tell me what value you have for: IAR_SYSTEMS_ICC' Thanks, |
Hi @SparkiDev The value I have for |
Hi @josepho0918, Thanks for the help! Sean |
19c6b28
to
a3a0247
Compare
Hi @SparkiDev, I have tested the latest changes, and IAR is complaining |
IAR doesn't like %=. Fix code to be consistent in use of labels and branch instructions.
a3a0247
to
3284f53
Compare
Hi @josepho0918, I've modified the code to use B*.W/N for IAR. Thanks, |
Hi @SparkiDev, Excellent work! There're no compilation errors now, and the tests have passed. |
@SparkiDev seeing consistent failures with GCC after this PR. Thanks @night1rider for spotting it. Can reproduce easily.
|
Description
IAR doesn't like %=.
Fix code to be consistent in use of labels and branch instructions.
Fixes zd#18315
Testing
./configure '--disable-shared' '--enable-all' '--enable-sp-asm' '--enable-armasm' 'LDFLAGS=--static' '--host=armv7m' 'CC=arm-linux-gnueabi-gcc' '--disable-crl-monitor'
./configure '--disable-shared' '--enable-all' '--enable-sp-asm' '--enable-armasm=inline' 'LDFLAGS=--static' '--host=armv7m' 'CC=arm-linux-gnueabi-gcc' '--disable-crl-monitor'
Checklist