RFR: 8199319: Remove handles.inline.hpp include from reflectionUtils.hpp

[David Holmes]

On 8/03/2018 11:59 PM, Stefan Karlsson wrote:
> I've now added include lines for handles.inline.hpp where needed:
> 
> http://cr.openjdk.java.net/~stefank/8199319/webrev.02.delta
> http://cr.openjdk.java.net/~stefank/8199319/webrev.02
> 
> The patch is a bit larger, but I still consider it trivial.

Looks fine.

Seeing this more explicit, I have to start to wonder how effective all 
this inlining actually is if we were to compare performance gains versus 
code bloat. ?? It also seems unfortunate that just because you want to 
inline one use of a function, that you have to inline them all (unless 
you define two versions of course).

Thanks,
David

> Thanks,
> StefanK
> 
> On 2018-03-08 10:36, Stefan Karlsson wrote:
>> Further testing on this with other build targets show that I need to 
>> add handles.inline.hpp includes to cpp files that used to get it 
>> included from reflectionUtils.hpp. I'll post a new webrev when I've 
>> run all build targets.
>>
>> StefanK
>>
>> On 2018-03-08 10:14, Stefan Karlsson wrote:
>>> Hi all,
>>>
>>> Please review this tiny patch to remove an unnecessary include of 
>>> handles.inline.hpp. This is a step towards getting cleaner include 
>>> dependencies.
>>>
>>> http://cr.openjdk.java.net/~stefank/8199319/webrev.01/
>>>
>>> I compiled this with gcc -E to look for indirect includes of 
>>> .inline.hpp files. While doing this I found that this file couldn't 
>>> be compiled stand-alone. So, I also added the missing 
>>> growableArray.hpp include.
>>>
>>> Thanks,
>>> StefanK
>>
>>
>