Generate c code cannot compile with recent devel version

[slangmgh]

With latest devel version ,the code can be compiled with -d:release, but not without release mode. The version about two days before is fine.

Example

# a.nim
import times
echo now()

Current Output

> nim c -d:release a.nim
Hint: 39170 LOC; 0.788 sec; 31.332MiB peakmem; Release build; proj: a; out: D:\Work\Stock\a.exe [SuccessX]

> nim c a.nim
CC: stdlib_times.nim                                                                                                  
stdlib_times.nim.c                                                                                                    
D:\Tmp\Temp\vcc\nimcache\d\stdlib_times.nim.c(310) : error C2059: syntax error : ','                                  
D:\Tmp\Temp\vcc\nimcache\d\stdlib_times.nim.c(310) : error C2143: syntax error : missing ')' before '('               
D:\Tmp\Temp\vcc\nimcache\d\stdlib_times.nim.c(310) : error C2091: function returns function                           
D:\Tmp\Temp\vcc\nimcache\d\stdlib_times.nim.c(310) : error C2059: syntax error : ')'                                  
D:\Tmp\Temp\vcc\nimcache\d\stdlib_times.nim.c(310) : error C2085: 'res' : not in formal parameter list                
D:\Tmp\Temp\vcc\nimcache\d\stdlib_times.nim.c(311) : error C2085: 'toBiggestFloat__hTpm9cXKgh17pxyZUsNnUyQsystem' : no
t in formal parameter list                                                                                            
D:\Tmp\Temp\vcc\nimcache\d\stdlib_times.nim.c(312) : error C2085: 'nimFrame' : not in formal parameter list           
.......

$ nim -v
Nim Compiler Version 1.1.1 [Windows: i386]
Compiled at 2020-03-14
Copyright (c) 2006-2019 by Andreas Rumpf

active boot switches: -d:release

[slangmgh]

relate to: Araq catchable defects (#13626)

[slangmgh]

Genereated code contain such function "nimMulInt64" have the same name with the macro in nimbase.h.

[Araq]

Cannot reproduce.

[juancarlospaco]

$ cat a.nim     
echo 42

$ nim c -f a.nim             
Hint: system [Processing]
Hint: widestrs [Processing]
Hint: io [Processing]
Hint: a [Processing]
CC: stdlib_io.nim
CC: stdlib_system.nim
CC: a.nim
Hint:  [Link]
/usr/bin/ld: /home/juan/.cache/nim/a_d/stdlib_io.nim.c.o: in function `echoBinSafe':
stdlib_io.nim.c:(.text+0x285): undefined reference to `nimAddInt'
/usr/bin/ld: /home/juan/.cache/nim/a_d/stdlib_system.nim.c.o: in function `newSeq':
stdlib_system.nim.c:(.text+0xbd50): undefined reference to `nimMulInt'
/usr/bin/ld: stdlib_system.nim.c:(.text+0xbd73): undefined reference to `nimAddInt'
/usr/bin/ld: /home/juan/.cache/nim/a_d/stdlib_system.nim.c.o: in function `atomicInc__v6qORsZ9azlHHhOL1JYACRAsystem':
stdlib_system.nim.c:(.text+0xd589): undefined reference to `nimAddInt'
collect2: error: ld returned 1 exit status
Error: execution of an external program failed: 'gcc   -o /home/juan/code/temp13/a  /home/juan/.cache/nim/a_d/stdlib_io.nim.c.o /home/juan/.cache/nim/a_d/stdlib_system.nim.c.o /home/juan/.cache/nim/a_d/@ma.nim.c.o    -ldl'

$ nim c -f  --nimcache:.  a.nim
Hint: system [Processing]
Hint: widestrs [Processing]
Hint: io [Processing]
Hint: a [Processing]
CC: stdlib_io.nim
CC: stdlib_system.nim
CC: a.nim
Hint:  [Link]
Hint: 14195 LOC; 0.630 sec; 16.027MiB peakmem; Debug build; proj: /home/juan/code/temp13/a.nim; out: /home/juan/code/temp13/a [SuccessX]

$

Not "codegen" tho.

With --nimcache:. it works, is not finding nimcache folder?, CFG ?.

[slangmgh]

need to include "import times".
To make code include function of integer add/div/mul.

[slangmgh]

The generated code:

static N_INLINE(NIM_BOOL, nimDivInt64)(NI64 a, NI64 b, NI64* res);
static N_INLINE(NIM_BOOL, nimMulInt64)(NI64 a, NI64 b, NI64* res);

And there is macro nimModInt64 in nimbase.h, so the c compiler cannot compile.

Turn off the option overflow_checks, these function will not exists, and it can be compiled correctly.

[juancarlospaco]

$ cat a.nim
discard

$ choosenim stable
   Switched to Nim 1.0.6

$ nim c -f a.nim
Hint: operation successful (14483 lines compiled; 0.621 sec total; 16.023MiB peakmem; Debug Build) [SuccessX]

$ choosenim devel 
   Switched to Nim #devel

$ nim c -f --listCmd --verbosity:2 a.nim
/home/juan/.choosenim/toolchains/nim-#devel/config/nim.cfg(48, 5) Hint: added path: '/opt/nimble/pkgs/' [Path]
      nimblepath="/opt/nimble/pkgs/"
      ^
/home/juan/.choosenim/toolchains/nim-#devel/config/nim.cfg(52, 3) Hint: added path: '/home/juan/.nimble/pkgs/jester-0.4.3' [Path]
    nimblepath="$home/.nimble/pkgs/"
    ^
/home/juan/.choosenim/toolchains/nim-#devel/config/nim.cfg(52, 3) Hint: added path: '/home/juan/.nimble/pkgs/nimpy-0.1.0' [Path]
    nimblepath="$home/.nimble/pkgs/"
    ^
/home/juan/.choosenim/toolchains/nim-#devel/config/nim.cfg(52, 3) Hint: added path: '/home/juan/.nimble/pkgs/' [Path]
    nimblepath="$home/.nimble/pkgs/"
    ^
Hint: used config file '/home/juan/.choosenim/toolchains/nim-#devel/config/nim.cfg' [Conf]
Hint: system [Processing]
Hint: widestrs [Processing]
Hint: io [Processing]
Hint: a [Processing]
CC: stdlib_system.nim: gcc -c  -w   -I'/home/juan/.choosenim/toolchains/nim-#devel/lib' -I/home/juan/code/temp13 -o /home/juan/.cache/nim/a_d/stdlib_system.nim.c.o /home/juan/.cache/nim/a_d/stdlib_system.nim.c
CC: a.nim: gcc -c  -w   -I'/home/juan/.choosenim/toolchains/nim-#devel/lib' -I/home/juan/code/temp13 -o /home/juan/.cache/nim/a_d/@ma.nim.c.o /home/juan/.cache/nim/a_d/@ma.nim.c
Hint: gcc   -o /home/juan/code/temp13/a  /home/juan/.cache/nim/a_d/stdlib_system.nim.c.o /home/juan/.cache/nim/a_d/@ma.nim.c.o    -ldl [Link]
/usr/bin/ld: /home/juan/.cache/nim/a_d/stdlib_system.nim.c.o: in function `newSeq':
stdlib_system.nim.c:(.text+0xbd50): undefined reference to `nimMulInt'
/usr/bin/ld: stdlib_system.nim.c:(.text+0xbd73): undefined reference to `nimAddInt'
/usr/bin/ld: /home/juan/.cache/nim/a_d/stdlib_system.nim.c.o: in function `atomicInc__v6qORsZ9azlHHhOL1JYACRAsystem':
stdlib_system.nim.c:(.text+0xd589): undefined reference to `nimAddInt'
collect2: error: ld returned 1 exit status
Error: execution of an external program failed: 'gcc   -o /home/juan/code/temp13/a  /home/juan/.cache/nim/a_d/stdlib_system.nim.c.o /home/juan/.cache/nim/a_d/@ma.nim.c.o    -ldl'

$ nim c -f --overflowChecks:off a.nim
/usr/bin/ld: /home/juan/.cache/nim/a_d/stdlib_system.nim.c.o: in function `newSeq':
stdlib_system.nim.c:(.text+0xbd50): undefined reference to `nimMulInt'
/usr/bin/ld: stdlib_system.nim.c:(.text+0xbd73): undefined reference to `nimAddInt'
collect2: error: ld returned 1 exit status
Error: execution of an external program failed: 'gcc   -o /home/juan/code/temp13/a  /home/juan/.cache/nim/a_d/stdlib_system.nim.c.o /home/juan/.cache/nim/a_d/@ma.nim.c.o    -ldl'

$ nim c -f --checks:off a.nim
/usr/bin/ld: /home/juan/.cache/nim/a_d/stdlib_system.nim.c.o: in function `newSeq':
stdlib_system.nim.c:(.text+0xbd50): undefined reference to `nimMulInt'
/usr/bin/ld: stdlib_system.nim.c:(.text+0xbd73): undefined reference to `nimAddInt'
collect2: error: ld returned 1 exit status
Error: execution of an external program failed: 'gcc   -o /home/juan/code/temp13/a  /home/juan/.cache/nim/a_d/stdlib_system.nim.c.o /home/juan/.cache/nim/a_d/@ma.nim.c.o    -ldl'

$ nim c -f --nimcache:. a.nim
Hint: 14195 LOC; 0.615 sec; 15.887MiB peakmem; Debug build; proj: /home/juan/code/temp13/a.nim; out: /home/juan/code/temp13/a [SuccessX]

$

[Araq]

as I said, can't reproduce.

[slangmgh]

Please try the code I given:

# a.nim
import times
echo now()

[slangmgh]

Or try this simple code.

var sum: int64
for x in 1..10:
   sum = sum + x

echo sum

Generated code include this line:

static N_INLINE(NIM_BOOL, nimAddInt64)(NI64 a, NI64 b, NI64* res);

Which is the problem, because there is a macro named nimAddInt64 in nimbase.h.

I using 32bit nim on windows, maybe the code only generated in this platform.

[Araq]

Well for me the code does not "include this line" and yes I'm checking on Windows 32 bit.

[Araq]

Ah! So it's VCC related, got it.