TDA4VEN-Q1: Code Compilation error for Eth driver

Hi,

Shubham Goel said:

We are using Compiler ti-cgt-armllvm_4.0.1.LTS, assembler doesn't recognize or support ";" for comments.

After updating "lld_arm_r5.asm", replacing ";" with "//" as per TI_Arm_Clang_Compiler_Tools_User_Guide(GNU-Syntax Comment Delimiters), we are able to fix the issue.

Yes, when you are using GNU syntax ";" is not a allowed comment style for asm files.

Shubham Goel said:

Could you please confirm the changes and provide your feedback on the same?

Changes are fine replaning ";" with "//".
In order to avould complation issue, you can use ti syntax style with "-x ti-asm" option as complier option or modify comment style in asm files with "//" insted ";".


Best Regards,
Sudheer

Hi,

Doredla Sudheer Kumar said:

Changes are fine replaning ";" with "//".
In order to avould complation issue, you can use ti syntax style with "-x ti-asm" option as complier option or modify comment style in asm files with "//" insted ";".

Thanks for the confirmation.

It works fine after replacing comment style ";" with "//".

But does not work with "-x ti-asm" as assembler option inclusion.

BR,
Shubham

Hi,

Sorry for delay in response as i missed the 2nd point somehow.

Doredla Sudheer Kumar said:

Can you confirm are you using the flag right before linking your assembler file

 "-x ti-asm file1.asm file2.asm file3.asm"

If you are having mix of gnu style and ti-style then you need to use as below. 

"-x ti-asm file1. asm -x none file2.s"

I tried to build single file lld_arm_r5.asm using the flag -x ti-asm right before the file and i was able to build it successfully.

I am debugging on test environment side why -x ti-asm is not considered for lld_arm_r5.asm.

I will let you know once there is any update.. In the mean time i am continuing with replacing comment style ";" with "/" for test execution.

BR,
Shubham

Hi, 

Shubham Goel said:

I tried to build single file lld_arm_r5.asm using the flag -x ti-asm right before the file and i was able to build it successfully

Thanks for confirming.

Shubham Goel said:

I will let you know once there is any update.. In the mean time i am continuing with replacing comment style ";" with "/" for test execution

Sure, mean time you can proceed with manually replacing the comment style. 

Best Regards, 

Sudheer

Hi,

On our test environment, flag -x assembler-with-cpp is enabled by default to interpret subsequent files as GNU-syntax Arm assembly source files that will be preprocessed by the compiler.

We will continue with replacing comment style ";" with "/" for GNU-syntax as a integration step.. No modification is needed from TI side.

Thanks a lot for your support during the discussion. We can close this thread.

BR,
Shubham

Hi,

Shubham Goel said:

On our test environment, flag -x assembler-with-cpp is enabled by default to interpret subsequent files as GNU-syntax Arm assembly source files that will be preprocessed by the compiler.

Understood, thanks for the details.

Shubham Goel said:

We will continue with replacing comment style ";" with "/" for GNU-syntax as a integration step.. No modification is needed from TI side.

You can go ahead with this manual change.

Best Regards,
Sudheer