RFR (S): C1 still does eden allocations when TLAB is enabled

[JC Beyler]

Yes, you are right, I did those changes due to:
https://bugs.openjdk.java.net/browse/JDK-8194084

If Robbin agrees to this change, and if no one sees an issue, I'll go ahead
and propagate the change across architectures.

Thanks for the review, I'll wait for Robbin (or anyone else's comment and
review) :)
Jc

On Fri, Jul 13, 2018 at 1:08 PM John Rose <john.r.rose at oracle.com> wrote:

> On Jul 13, 2018, at 10:23 AM, JC Beyler <jcbeyler at google.com> wrote:
>
>
> I'm not sure if we had left this case intentionally or not but, if we want
> it all to be consistent, we should perhaps fix it.
>
>
> Well, you put in that logic last February, so unless somebody speaks up
> quickly, I support your adjusting it to be the way you want it.
>
> Doing "hg grep -u supports_inline_contig_alloc -I src/hotspot/share"
> suggests that the GC group is most active in touching this feature.
> If Robbin is OK with it, there's your reviewer.
>
> FWIW, you can use me as a reviewer, but I'd get one other person
> working on the GC to OK it.
>
> — John
>


-- 

Thanks,
Jc