[sdcc-devel] [ sdcc-Bugs-3508516 ] asm

[SourceForge.net <no...@so...>]

Bugs item #3508516, was opened at 2012-03-19 02:55
Message generated for change (Comment added) made by borutr
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100599&aid=3508516&group_id=599

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: C-Front End
>Group: fixed
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Philipp Klaus Krause (spth)
Assigned to: Borut Ražem (borutr)
Summary: asm

Initial Comment:
Trying to compile the follwoing function fails with error message "syntax error: token -> 'asm' ; column 8" as of sdcc revision #7469.

void asm (void)
{
}

AFAIK this is a regression that can break existing code.

Philipp


----------------------------------------------------------------------

>Comment By: Borut Ražem (borutr)
Date: 2012-03-19 13:39

Message:
asm() removed in svn revision #7471.

Borut

----------------------------------------------------------------------

Comment By: Philipp Klaus Krause (spth)
Date: 2012-03-19 10:56

Message:
I strongly think, that no keywords / macros / functions in standard headers
should be introduced by sdcc outside of what the standard allows. People
had problems for years due to non-compliant keywords (especially "data") in
sdcc, when trying to compile their C programs. I think sdcc should move
closer to standard-compliance, not further from it.
Removing the asm, and using only the standard-compliant __asm__ looks like
the easiest way to resolve this issue to me.

Philipp


----------------------------------------------------------------------

Comment By: Borut Ražem (borutr)
Date: 2012-03-19 10:48

Message:
asm and __asm__ are now reserved keywords, see:
https://sourceforge.net/tracker/?func=detail&atid=350599&aid=2815320&group_id=599

Any suggestion how to handle this problem, if it should be handled at all?

One option is to remove asm and leave only __asm__ as a reserved
keyword...

Borut

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100599&aid=3508516&group_id=599