RFR: 8058716: Add include missing in 8015774

[Lindenmaier, Goetz]

Hi Vladimir, 

thanks for reviewing & pushing this!  

Best regards,
  Goetz

-----Original Message-----
From: hotspot-dev [mailto:hotspot-dev-bounces at openjdk.java.net] On Behalf Of Vladimir Kozlov
Sent: Donnerstag, 18. September 2014 22:25
To: hotspot-dev at openjdk.java.net
Subject: Re: RFR: 8058716: Add include missing in 8015774

On 9/18/14 1:07 AM, Lindenmaier, Goetz wrote:
> Hi,
>
> 8015744 breaks the build on ppc as an inline function is not defined:

   8015774


> compile.hpp:812: warning: inline function 'Node_Notes* Compile::locate_node_notes(GrowableArray<Node_Notes*>*, int, bool)' used but never defined
>
> This is because compile.hpp declares and calls locate_node_notes(), which is defined in node.hpp. Therefore codeCache.cpp must include node.hpp, too.

Good.

Thanks,
Vladimir

>
> Bug:
> https://bugs.openjdk.java.net/browse/JDK-8058716
> Webrev:
> http://cr.openjdk.java.net/~goetz/webrevs/8058716-inclFix/webrev.00/
>
> Please review this change.  I please need a sponsor to push it.
>
> Best regards,
>    Goetz
>