HotSpot build failure with GCC 7.1.1

[Yasumasa Suenaga]

Hi David,

> I'm still at a loss to understand how the current code even compiles/links
> at all with any version of the compiler. ??

I guess Array<T>::size() is not used from any code.
For example, clang 4.0.0 in Fedora 26 passes this compilation, however
GCC 7.1.1 is failed.

I grep'ed following command, but I could not find the use.
--------------
grep -nrE 'Array.+align_size_up' *
--------------

Most compilers might not check unused declarations.


> You removed a "// FIXME" comment - what was that referring to ??

Sorry, I missed.
I guess the reason of "FIXME" is known by the author of JDK-8072061.


Thanks,

Yasumasa


2017-08-04 11:00 GMT+09:00 David Holmes <david.holmes at oracle.com>:
> Hi Yasumasa,
>
> On 4/08/2017 10:49 AM, Yasumasa Suenaga wrote:
>>
>> Thanks Kim, David,
>>
>> I filed this issue to JBS and uploaded webrev. Could you review it?
>>
>>    http://cr.openjdk.java.net/~ysuenaga/JDK-8185826/webrev.00/
>
>
> I'm still at a loss to understand how the current code even compiles/links
> at all with any version of the compiler. ??
>
> You removed a "// FIXME" comment - what was that referring to ??
>
> Thanks,
> David
>
>
>> I cannot access JPRT. So I need a sponsor.
>>
>>
>> Yasumasa
>>
>>
>>
>> 2017-08-04 9:08 GMT+09:00 Kim Barrett <kim.barrett at oracle.com>:
>>>>
>>>> On Aug 3, 2017, at 7:12 PM, David Holmes <david.holmes at oracle.com>
>>>> wrote:
>>>> I can't even see a definition of align_size_up - where is it coming from
>>>> ???
>>>
>>>
>>> It was recently removed.
>>>
>>> 8178499: Remove _ptr_ and _size_ infixes from align functions
>>>
>