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