Skip to content

Conversation

@nijtmans
Copy link
Collaborator

When compiling release/1.2.0 with latest Visual Studio (2019), there were 2 new error-messages which resulted in a failed build:

C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.22.27905\include\vcruntime.h(255): error C2220: warning treated as error - no 'object' file generated
C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.22.27905\include\vcruntime.h(255): warning C4668: '__cplusplus' is not defined as a preprocessor macro, replacing with '0' for '#if/#elif'
C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.22.27905\include\vcruntime.h(258): warning C4668: '__cplusplus' is not defined as a preprocessor macro, replacing with '0' for '#if/#elif'
NMAKE : fatal error U1077: '"C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.22.27905\bin\HostX64\x64\cl.EXE"' : return code '0x2'
S
.....
C:\Users\jan.nijtmans\workspace\libtommath\bn_deprecated.c(12) : error C2220: warning treated as error - no 'object' file generated
C:\Users\jan.nijtmans\workspace\libtommath\bn_deprecated.c(12) : warning C5045: Compiler will insert Spectre mitigation for memory load if /Qspectre switch specified
C:\Users\jan.nijtmans\workspace\libtommath\bn_deprecated.c(9) : note: index 'b' range checked by comparison on this line
C:\Users\jan.nijtmans\workspace\libtommath\bn_deprecated.c(12) : note: feeds call on this line
C
....

Clearly those are errors in visual studio :-(

@sjaeckel sjaeckel force-pushed the fix-msvc-3 branch 2 times, most recently from b1d0168 to bd4f131 Compare October 18, 2019 13:40
@sjaeckel
Copy link
Member

can I merge?

@sjaeckel sjaeckel merged commit 192fad3 into release/1.2.0 Oct 18, 2019
@sjaeckel sjaeckel deleted the fix-msvc-3 branch October 18, 2019 13:55
@sjaeckel sjaeckel removed the finished label Oct 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants