nbind icon indicating copy to clipboard operation
nbind copied to clipboard

Doesn't compile with latest Visual Studio 2017 update

Open kmkim85 opened this issue 7 years ago • 5 comments

With VS Update 15.5.2, existing project using nbind stopped compiling.

\nbind\include\nbind\v8/Caller.h(68): fatal error C1001: An internal error has occurred in the compiler.

kmkim85 avatar Dec 15 '17 21:12 kmkim85

Nice... That's clearly a new compiler bug. Try commenting that line out, does it work then?

It's checking if some input parameters are valid and can be omitted if your JavaScript side code is correct anyway. Then I can rewrite that line differently. However if there's other new compiler crashes all over the place, that will be a different matter.

jjrv avatar Dec 16 '17 10:12 jjrv

Commenting out the line didn't generate further compiler issues

kmkim85 avatar Dec 19 '17 18:12 kmkim85

OK, thanks for reporting! I'll install the latest VS Community edition so the bug would hopefully appear here, and then try to figure out an alternative way to write that line. Until then, at least locally you can use the commented out version.

jjrv avatar Dec 20 '17 12:12 jjrv

Hi, Any updates on fixing this issue? We would really like a fix soon as this is preventing us from moving to VS 2017.

kaushiksanthanam avatar Feb 23 '18 20:02 kaushiksanthanam

Sorry about the delay. 0.3.15 is out now and at least on my machine, the compiler internal error is fixed.

jjrv avatar Mar 02 '18 13:03 jjrv