Name clashes - was: Re: [RFR]: 8186723: Add SuperH as new architecture for linux

David Holmes david.holmes at oracle.com
Thu Aug 24 13:07:33 UTC 2017


On 24/08/2017 9:21 PM, John Paul Adrian Glaubitz wrote:
> On 08/24/2017 12:28 PM, Thomas Stüfe wrote:
>> > On Thu, Aug 24, 2017 at 12:05 PM, John Paul Adrian Glaubitz 
>> <glaubitz at physik.fu-berlin.de <mailto:glaubitz at physik.fu-berlin.de>> 
>> wrote:
>> > > On 08/24/2017 11:57 AM, Thomas Stüfe wrote:
>>
>>         Ah, I see it. "sh" was introduced by you in JDK-8186313 in 
>> platform.m4 (VAR_CPU).
>>         I just wondered why this macro was so shortish and whether we 
>> would risk potential
>>         name clashes with other macros.
>>
>>     I wanted to discuss this in a separate issue.
>>
>> Sure.
> 
> Ok, so let's continue here as I'm not sure whether this issue should be 
> addressed
> before merging the above changeset.  In my opinion, it's generally 
> rather suboptimal
> approach to use any global defines without surrounding underscores. But 
> I am currently
> undecided what would be the best approach to resolve this problem.

Sorry I don't have any context here. What is the issue? And is it a 
hotspot issue or a build issue?

Thanks,
David

> Adrian
> 


More information about the hotspot-dev mailing list