RFR(xxs) 8074543 Missing symbol "objArrayOopDesc::obj_at" when buiding with CPP Interpreter

[Stefan Karlsson]

Hi Thomas,

On 06/03/15 14:28, Thomas Stüfe wrote:
> Hi Stefan,
>
> here you go:
>
> http://cr.openjdk.java.net/~stuefe/webrevs/8074543/webrev.02/webrev 
> <http://cr.openjdk.java.net/%7Estuefe/webrevs/8074543/webrev.02/webrev>
>
> I also would need a sponsor, because I am no author for the hotspot 
> project.

There's no need for a second reviewer, given the size of the patch. I'll 
push this to jdk9/hs-gc/hotspot.

Thanks,
StefanK

>
> Thanks, Thomas
>
>
>
>
> On Fri, Mar 6, 2015 at 2:15 PM, Stefan Karlsson 
> <stefan.karlsson at oracle.com <mailto:stefan.karlsson at oracle.com>> wrote:
>
>     Hi Thomas,
>
>     On 2015-03-06 12:15, Thomas Stüfe wrote:
>
>         Hi all,
>
>         please review this tiny change to fix a build error when
>         building with the
>         CPP interpreter.
>
>         webrev:
>         http://cr.openjdk.java.net/~stuefe/webrevs/8074543/webrev.01/
>         <http://cr.openjdk.java.net/%7Estuefe/webrevs/8074543/webrev.01/>
>
>
>      #include "oops/objArrayKlass.hpp"
>      #include "oops/oop.inline.hpp"
>     +#include "oops/objArrayOop.inline.hpp"
>
>
>     I'm fine with the change if the added line is moved up one line so
>     that we keep the includes sorted.
>
>     Thanks,
>     StefanK
>
>
>
>         bug:
>         https://bugs.openjdk.java.net/browse/JDK-8074543
>
>         bytecodeInterpreter.cpp misses include
>         "objArrayOop.inline.hpp", so the
>         inline definition of this function is missing.
>
>         Kind Regards, Thomas
>
>
>