request for review - 6806046 (XXS)

Ivan Krylov Ivan.Krylov at Sun.COM
Wed Feb 18 01:06:12 PST 2009


This is a fix for CR 6806046.

There is a file that we keep in the repository called create.bat
This file creates a Visual Studio project using java code from share\tools\MakeDeps
At some point compilation of vm got broken in such generated projects.

This is the error message:
src\share\vm\compiler\disassembler.cpp(34) : error C2146: syntax error : missing ';' before identifier 'HOTSPOT_LIB_ARCH'
src\share\vm\compiler\disassembler.cpp(34) : fatal error C1004: unexpected end of file found

Clearly it is because HOTSPOT_LIB_ARCH is not defined in the preprocessor definitions if the project properties.

Since we do not use the IDE for production binaries this functionality is secondary.

Still this 1-line fix is worth integrating, imo.

=== *Suggested Fix* ==========================================================
--- old/src/share/tools/MakeDeps/    Mon Feb 16 18:25:13 2009
+++ new/src/share/tools/MakeDeps/    Mon Feb 16 18:25:13 2009
@@ -247,6 +247,7 @@
+        sysDefines.add("HOTSPOT_LIB_ARCH=\"i486\"");



PS There is a file to create a VS project for 64-bit vm but I was not updated for years. I will look into it later.

More information about the hotspot-runtime-dev mailing list