From: SourceForge.net <no...@so...> - 2012-03-19 20:39:29
|
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 |