Re: [Tinycc-devel] PPTest13 issue on RPi

[Christian Jullien]

Hello Michael,

Thank you for your time and your fix.
Trying your patch (i.e. w.o. #if test) makes all PPtest work.

You can commit it.

C.

-----Original Message-----
From: Michael Matz [mailto:address@hidden 
Sent: lundi 3 octobre 2016 18:38
To: Christian JULLIEN; tinycc-devel
Subject: Re: [Tinycc-devel] PPTest13 issue on RPi

Hi,

On Mon, 3 Oct 2016, Christian JULLIEN wrote:

> On RPI, for monthes now, I have this error:
> address@hidden:~/tinycc/tests/pp $ ../../tcc -E -P 13.S >13.output 2>&1 
> ; diff -Nu -b -B -I "^#" ../../tests/pp/13.expect 13.output
> --- ../../tests/pp/13.expect    2016-10-03 13:15:02.292455721 +0200
> +++ 13.output    2016-10-03 13:22:36.991074604 +0200
> @@ -1,6 +1,2 @@
> -# `modelist' label. Each video mode record looks like:
>  
> -
> -
> -.text
> -endtext:
> +13.S:0: error: unrecognized character \x60
> 
> => +13.S:0: error: unrecognized character \x60 is because of backquote 
> before m in the first line: "# `modelist' label. Each video mode 
> record looks like:"
> 
> address@hidden:~/tinycc/tests/pp $ more 13.output
> 
> 13.S:0: error: unrecognized character \x60
> 
> 
> However, it works well on Linux x86_64

Try the below.


Ciao,
Michael.

diff --git a/libtcc.c b/libtcc.c
index 8e1ed05..dde24b8 100644
--- a/libtcc.c
+++ b/libtcc.c
@@ -1282,13 +1282,11 @@ ST_FUNC int tcc_add_file_internal(TCCState *s1, const 
char *filename, int flags,
      int fd, ret, size;

      set_parse_flags (0);
-#ifdef CONFIG_TCC_ASM
      /* if .S file, define __ASSEMBLER__ like gcc does */
      if ((filetype == TCC_FILETYPE_ASM) || (filetype == TCC_FILETYPE_ASM_PP)) {
          tcc_define_symbol(s1, "__ASSEMBLER__", NULL);
          set_parse_flags (PARSE_FLAG_ASM_FILE);
      }
-#endif

      /* open the file */
      ret = tcc_open(s1, filename);