[sdcc-devel] [ sdcc-Bugs-3515390 ] asm with leading whitespace passed to backends

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

Bugs item #3515390, was opened at 2012-04-05 23:31
Message generated for change (Comment added) made by woody1234
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100599&aid=3515390&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: common target code generation
Group: fixed
Status: Closed
Resolution: Fixed
Priority: 5
Private: No
Submitted By: Woody (woody1234)
Assigned to: Borut Ražem (borutr)
Summary:  asm with leading whitespace passed to backends

Initial Comment:
1. Sample code attached.
2. Command line used: C:\SDCC\BIN\sdcc isr.c -mz80 -c --std-c99 --codeseg CODE0
3. SDCC version:  SDCC : mcs51/gbz80/z80/z180/r2k/ds390/pic16/pic14/TININative/ds400/hc08 3.1.4 #7
531 (Apr  4 2012) (MINGW32)
4.  Generated following warnings:
C:\sdcc\bug>C:\SDCC\BIN\sdcc isr.c -mz80 -c --std-c99 --codeseg CODE0
Warning: z80instructionSize() failed to parse line node  nop
Warning: z80instructionSize() failed to parse line node  nop
Warning: z80instructionSize() failed to parse line node  push iy
Warning: z80instructionSize() failed to parse line node   push af
Warning: z80instructionSize() failed to parse line node  exx
Warning: z80instructionSize() failed to parse line node  push iy
Warning: z80instructionSize() failed to parse line node   push af
Warning: z80instructionSize() failed to parse line node  exx
Warning: z80instructionSize() failed to parse line node  push iy
Warning: z80instructionSize() failed to parse line node   push af
Warning: z80instructionSize() failed to parse line node  exx
Warning: z80instructionSize() failed to parse line node  push iy
Warning: z80instructionSize() failed to parse line node   push af
Warning: z80instructionSize() failed to parse line node  exx
Warning: z80instructionSize() failed to parse line node  push iy
Warning: z80instructionSize() failed to parse line node   push af
Warning: z80instructionSize() failed to parse line node  exx
Warning: z80instructionSize() failed to parse line node  push iy
Warning: z80instructionSize() failed to parse line node   push af
Warning: z80instructionSize() failed to parse line node  exx
Warning: z80instructionSize() failed to parse line node  push iy
Warning: z80instructionSize() failed to parse line node   push af
Warning: z80instructionSize() failed to parse line node  exx
Warning: z80instructionSize() failed to parse line node  push iy
Warning: z80instructionSize() failed to parse line node   push af
Warning: z80instructionSize() failed to parse line node  exx
Warning: z80instructionSize() failed to parse line node  push iy
Warning: z80instructionSize() failed to parse line node   push af
Warning: z80instructionSize() failed to parse line node  exx


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

Comment By: Woody (woody1234)
Date: 2012-04-08 04:12

Message:
I tested #7546 SDCC :
mcs51/gbz80/z80/z180/r2k/ds390/pic16/pic14/TININative/ds400/hc08 3.1.4
#7546 (Apr  7 2012) (MINGW32)
Seems that the problem is still there, with a new signal 11:
C:\SDCC\BIN\sdcc isr.c -mz80 -c --std-c99 --codeseg CODE0
Warning: z80instructionSize() failed to parse line node  nop
Warning: z80instructionSize() failed to parse line node  nop
Caught signal 11: SIGSEGV
..\bin\make: *** [isr.rel] Error 1

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

Comment By: Borut Ražem (borutr)
Date: 2012-04-06 10:56

Message:
Fixed in svn revision #7543.

Woody, can you please verify it?

Borut

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

Comment By: Philipp Klaus Krause (spth)
Date: 2012-04-06 08:20

Message:
This is not a z80-specific bug: The backends should receveive asm lines
without leading whitespace for their instruction size functions (currently
the z80-related ports, ds390 and mcs51 have one). AFAIK all these functions
assume there to be no leading whitespace. However it seems sdcc fails to
strip the leading whitespace if the lines come from inline asm.

Workaround until this is fixed: Do not use leading whitespace on inline asm
lines.

Philipp


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

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