RFR(L): 8161259: Simplify including platform files.

Andrew Haley aph at redhat.com
Thu Jul 14 09:20:13 UTC 2016


On 14/07/16 10:17, Lindenmaier, Goetz wrote:
> Hi Andrew, 
> 
> thanks for looking at this right away.
> 
>>> Please review this webrev.
>>
>> Mostly looks fine, but I don't like that we can't grep to see where
>> files get included.
> Well, you need to grep for "cpu_header(assembler".

Well, yes, you do, but the point is *you don't know that*.  You're
a maintenance programmer, trying to fix a bug.

>> Are we really happy with
>>   40 #define SUB(x) x
>>
>> SUB is a very generic name, and this is some very weird macro hacking.
>> At least we need an explanation in a comment.
> 
> That's a good point, I'll rename it to HSSUB (hotspot submacro).
> It should not be too long because it's used so often on one line.
> 
> Comment:
> // Used to note down several macros without spaces to separate 
> // them:  For M1=foo and M2=bar, the preprocessor does not 
> // understand M1M2, but SUB(M1)M2 will work.

OK, so it's a neat way to concatenate macros.

Andrew.




More information about the hotspot-compiler-dev mailing list