Review request: 8006878 Some non-existent GC source files are in the minimalVM exclude list

Joseph Provino joseph.provino at oracle.com
Wed Feb 6 16:06:37 PST 2013


On 02/06/2013 06:56 PM, David Holmes wrote:
> On 7/02/2013 12:23 AM, Joe Provino wrote:
>> On 1/31/2013 3:25 AM, David Holmes wrote:
>>>
>>> On 26/01/2013 2:28 AM, Joe Provino wrote:
>>>> 2 ffiles changed.
>>>>
>>>> Webrev is here: http://cr.openjdk.java.net/~jprovino/8006878/webrev.00
>>>
>>> This seems to mix fixes for two different issues. The removal of the
>>> INCLUDE_ALL_GC guard would seem to indicate there is a problem if not
>>> building all GCs ie with the minimal VM - is that the case?
>>
>> David, sorry, I don't know how I'm managing to miss responses.
>>
>> I think the problem in excludeSrc.make excluding the non-existent file
>> "yieldWorkingGroup.cpp"
>> instead of the correct file "yieldingWorkGroup.cpp" meant that
>> yieldingWorkGroup.cpp
>> wasn't being excluded and didn't compile so I put in the guard to fix 
>> it.
>>
>> However, that being said, I tried not removing yieldingWorkGroup.cpp
>> when INCLUDE_ALL_GCS is 0 and it compiled. So if there was a problem it
>> doesn't appear to be a problem any more.
>
> Thanks Joe. Thumbs up! I had missed the fact that one of the changes 
> was an error in the file name. Once it is properly excluded we 
> certainly don't need the additional guard inside it, so that is fine. 
> (Aside: did you try your experiment with precompiled headers disabled?)

I didn't.  That might have been the problem.  It might have failed with 
JPRT on a Solaris build
which doesn't use precompiled headers.

joe

>
> I wonder if we could add a validation check for those filelists? 
> Probably tricky without the paths. Future work anyway not needed for 
> this CR.
>
> David
>
>
>
>>
>> joe
>>>
>>> David
>>>
>>>> joe



More information about the hotspot-dev mailing list