Compiler/MSP430FR2633: Compilation Error: error #18: expected a ")"

I'm not sure I understand what environment you're using, but: Have you tried capturing the preprocessor output? (-E I think) It's a bit cumbersome to work with, but it will tell you exactly what the compiler (proper) is seeing.

I have not...but I'll give it a shot!

`-E` didn't appear to be do anything.

Also tried `--gen_preprocessor_listing` which didn't appear to create an .rl file as it said it would?

OK, got an .rl file.  See attached.

The line in question in this file is line 42960, __bis_SR_register(GIE).  If you look, it replaces that line with the following:

__asm__ __volatile__ ("bis.w %0, SR { nop" : : "ri"((unsigned int) (0x0008)) );

Which from what I can tell...it's missing an "}" somewhere?

main.txt

So I've discovered the problem.  If I download the library files online from here: 

I would expect to see something like "\r\n" (without the quotes) where the "{" is.

This seems to happen throughout "/msp430-gcc-support-files/include/in430.h" (rev 1.205) [Ref main.txt line 2665 (ff)]

Do you know the origin of this .h file?

I'm just pulling it down from TI's website here:  http://software-dl.ti.com/msp430/msp430_public_sw/mcu/msp430/MSPGCC/6_1_1_0/index_FDS.html

I'm downloading the "gcc support files", unzipping, and it is in the /include/ folder.

[Hmm, you beat me to it.]

This appears to be a thing in CCS (v8.3.0) as well.

ccsv8/ccs_base/msp430/include_gcc/in430.h also has this typo, but only for "HAS_MSP430XV2_CPU" or "HAS_MSP430X_CPU". It's correct in the "#else" part.

This looks like a typo(s) to me, but I should defer to the Wizards here [Anyone?] since I'm not much familiar with GCC for the MSP430.

How do we signal the wizards?

Hm. Might work. TI monitors the Forum.

In the meantime, you could try hand-editing, since the damage appears to be localized. Scanning around it looks like you want "\n\t" in place of the "{".

Hi Brad,

Please provide more details about your development environment and source code so we can reproduce you issue here.

Thanks,

Ling

Hi Brad,

Thanks a lot. I will forward your feedback to MSP430 compiler team.

Ling

Hi,

The curly brace "{" is a GNU assembler construct that gets interpreted as a new line. There's a bit more info here: https://e2e.ti.com/support/tools/ccs/f/81/p/503224/1822907#1822907

The msp430-gcc-support-files are only intended for use with MSP430-GCC, there should be alternatives available for the TI compiler somewhere.

Regards,

As far as I can tell, this is the only issue with using the files with the MSP430 compiler.  Suggest fixing it to be compatible with both if it's the only thing that's not compatible.  Very confusing to have two different versions of the exact same file (in430.h) and claiming that only some versions are compatible with some compilers.