github-repos/tinycc
1
0
Fork 0
mirror of https://github.com/mirror/tinycc.git synced 2025-04-01 12:30:08 +08:00
Code Issues Packages Projects Releases Wiki Activity

(bug caused by tcc intristics)

Browse Source 
reproduce bug:
$ ./configure --cc=gcc
$ make
$ make install
(OK)
run a test:

extern int printf(const char *str, ...);
int main()
{
        int t2 = (int)(-1.847759065f * 4096);
        printf("%d\n", t2);
}

$ tcc test.c
$ ./a.out
$ -7568
(OK)

(self compiled now)
$ ./configure --cc=tcc
$ make
$ make install
(OK)

$ tcc test.c
$ ./a.out
$ 7568
(WRONG!!!)

why:
gcc does not have intristics for
uint to long double conversion
therefore it does cast implicitly, so
the sign bit is preserved, but this does
not happen when __fixunsxfdi is called
because tcc was bootstrapped.

solution:
force cast to int64 and preserve the
sign bit.

side effects:
not found.
This commit is contained in:
Kyryl Melekhin 2020-09-08 22:12:01 +00:00
parent 55f8963dfa
commit 310e3b428c
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tccgen.c
View File

@ -3478,7 +3478,7 @@ again:
vtop->c.i = (vtop->c.ld != 0); vtop->c.i = (vtop->c.ld != 0);
} else { } else {
if(sf) if(sf)
vtop->c.i = vtop->c.ld; vtop->c.i = (int64_t)vtop->c.ld;
else if (sbt_bt == VT_LLONG || (PTR_SIZE == 8 && sbt == VT_PTR)) else if (sbt_bt == VT_LLONG || (PTR_SIZE == 8 && sbt == VT_PTR))
; ;
else if (sbt & VT_UNSIGNED) else if (sbt & VT_UNSIGNED)