RFR (XXS): 8144521: [JVMCI] JVMCI is built on 32-bit Windows compiler2 and tiered builds

Christian Thalinger christian.thalinger at oracle.com
Wed Dec 2 19:43:29 UTC 2015


Hmm.  Looks like we also need a change like this:

diff -r 8578909eeef4 make/windows/create_obj_files.sh
--- a/make/windows/create_obj_files.sh	Thu Nov 26 10:38:33 2015 +0000
+++ b/make/windows/create_obj_files.sh	Wed Dec 02 09:42:23 2015 -1000
@@ -129,7 +129,7 @@ esac
 
 # Special handling of arch model.
 case "${Platform_arch_model}" in
-	"x86_32") Src_Files_EXCLUDE="${Src_Files_EXCLUDE} *x86_64*" ;;
+	"x86_32") Src_Files_EXCLUDE="${Src_Files_EXCLUDE} ${JVMCI_SPECIFIC_FILES} *x86_64*" ;;
 	"x86_64") Src_Files_EXCLUDE="${Src_Files_EXCLUDE} *x86_32*" ;;
 esac
 

> On Dec 2, 2015, at 8:43 AM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
> 
> Looks good.
> 
> Vladimir
> 
> On 12/2/15 9:58 AM, Christian Thalinger wrote:
>> https://bugs.openjdk.java.net/browse/JDK-8144521
>> 
>> Currently we build JVMCI for 32-bit Windows targets but we shouldn’t.
>> 
>> diff -r 8578909eeef4 make/windows/makefiles/vm.make
>> --- a/make/windows/makefiles/vm.makeThu Nov 26 10:38:33 2015 +0000
>> +++ b/make/windows/makefiles/vm.makeWed Dec 02 07:41:29 2015 -1000
>> @@ -45,10 +45,16 @@ CXX_FLAGS=$(CXX_FLAGS) /D "COMPILER1" /D
>> 
>> 
>>  !if "$(Variant)" == "compiler2"
>>  CXX_FLAGS=$(CXX_FLAGS) /D "COMPILER2"
>> +!if "$(BUILDARCH)" == "i486"
>> +CXX_FLAGS=$(CXX_FLAGS) /D INCLUDE_JVMCI=0
>> +!endif
>>  !endif
>> 
>> 
>>  !if "$(Variant)" == "tiered"
>>  CXX_FLAGS=$(CXX_FLAGS) /D "COMPILER1" /D "COMPILER2"
>> +!if "$(BUILDARCH)" == "i486"
>> +CXX_FLAGS=$(CXX_FLAGS) /D INCLUDE_JVMCI=0
>> +!endif
>>  !endif
>> 
>> 
>>  !if "$(BUILDARCH)" == "i486"
>> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20151202/90849ed3/attachment.html>


More information about the hotspot-compiler-dev mailing list