RFR(XS) 8239854: Cannot build gtest without PCH after 8239235. Missing include in test/hotspot/gtest/runtime/test_signatureStream.cpp
Doerr, Martin
martin.doerr at sap.com
Mon Feb 24 11:05:08 UTC 2020
Hi Richard,
thanks for the quick response. Yes, looks good.
Best regards,
Martin
> -----Original Message-----
> From: Reingruber, Richard <richard.reingruber at sap.com>
> Sent: Montag, 24. Februar 2020 12:03
> To: Aleksey Shipilev <shade at redhat.com>; Doerr, Martin
> <martin.doerr at sap.com>; Claes Redestad <claes.redestad at oracle.com>;
> hotspot-runtime-dev at openjdk.java.net
> Subject: RE: RFR(XS) 8239854: Cannot build gtest without PCH after 8239235.
> Missing include in test/hotspot/gtest/runtime/test_signatureStream.cpp
>
> Thanks Aleksey and Martin,
>
> I changed the synopsis and sorted the includes, but kept the newline at the
> end of the file.
>
> Are you ok with it?
>
> Thanks, Richard.
>
> diff --git a/test/hotspot/gtest/runtime/test_signatureStream.cpp
> b/test/hotspot/gtest/runtime/test_signatureStream.cpp
> --- a/test/hotspot/gtest/runtime/test_signatureStream.cpp
> +++ b/test/hotspot/gtest/runtime/test_signatureStream.cpp
> @@ -22,8 +22,9 @@
> */
>
> #include "precompiled.hpp"
> +#include "classfile/symbolTable.hpp"
> +#include "memory/resourceArea.hpp"
> #include "runtime/interfaceSupport.inline.hpp"
> -#include "classfile/symbolTable.hpp"
> #include "runtime/signature.hpp"
> #include "threadHelper.inline.hpp"
> #include "unittest.hpp"
> @@ -85,4 +86,4 @@
>
> ASSERT_EQ(foo->refcount(), r1) << "refcount should have decremented";
>
> -}
> \ No newline at end of file
> +}
>
> -----Original Message-----
> From: Aleksey Shipilev <shade at redhat.com>
> Sent: Montag, 24. Februar 2020 11:41
> To: Reingruber, Richard <richard.reingruber at sap.com>; Claes Redestad
> <claes.redestad at oracle.com>; hotspot-runtime-dev at openjdk.java.net
> Subject: Re: RFR(XS) 8239854: Cannot build gtest without PCH after 8239235.
> Missing include in test/hotspot/gtest/runtime/test_signatureStream.cpp
>
> On 2/24/20 11:35 AM, Reingruber, Richard wrote:
> > diff --git a/test/hotspot/gtest/runtime/test_signatureStream.cpp
> b/test/hotspot/gtest/runtime/test_signatureStream.cpp
> > --- a/test/hotspot/gtest/runtime/test_signatureStream.cpp
> > +++ b/test/hotspot/gtest/runtime/test_signatureStream.cpp
> > @@ -22,6 +22,7 @@
> > */
> >
> > #include "precompiled.hpp"
> > +#include "memory/resourceArea.hpp"
> > #include "runtime/interfaceSupport.inline.hpp"
> > #include "classfile/symbolTable.hpp"
> > #include "runtime/signature.hpp"
> > @@ -85,4 +86,4 @@
> >
> > ASSERT_EQ(foo->refcount(), r1) << "refcount should have
> decremented";
> >
> > -}
> > \ No newline at end of file
> > +}
>
> Looks good and trivial.
>
> Maybe drop the newline hunk? I have no strong preference here.
>
> Maybe synopsis could be a bit shorter?
> "Non-PCH gtest build fails after JDK-8239235 due to a missing include"
>
> --
> Thanks,
> -Aleksey
More information about the hotspot-runtime-dev
mailing list