RFR(L): 8161259: Simplify including platform files.
Lindenmaier, Goetz
goetz.lindenmaier at sap.com
Thu Jul 14 09:17:04 UTC 2016
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".
> 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.
Best regards,
Goetz.
>
> Andrew.
More information about the hotspot-compiler-dev
mailing list