# HG changeset patch # User kbarrett # Date 1552284307 14400 # Node ID 744dc9c336762662e5c60d0fb45ebf91cc47ac35 # Parent 0a9cb99a98a0d786ea46a21186d5495d171aaeb1 8217417: Decorator name typo: C2_TIGHLY_COUPLED_ALLOC Summary: Fixed typo in decorator name, variables, and comments. Reviewed-by: tschatzl diff -r 0a9cb99a98a0 -r 744dc9c33676 src/hotspot/share/gc/shared/c2/barrierSetC2.hpp --- a/src/hotspot/share/gc/shared/c2/barrierSetC2.hpp Mon Mar 11 00:06:48 2019 +0100 +++ b/src/hotspot/share/gc/shared/c2/barrierSetC2.hpp Mon Mar 11 02:05:07 2019 -0400 @@ -1,5 +1,5 @@ /* - * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it @@ -51,7 +51,7 @@ // This denotes that the access reads state. const DecoratorSet C2_READ_ACCESS = DECORATOR_LAST << 8; // A nearby allocation? -const DecoratorSet C2_TIGHLY_COUPLED_ALLOC = DECORATOR_LAST << 9; +const DecoratorSet C2_TIGHTLY_COUPLED_ALLOC = DECORATOR_LAST << 9; // Loads and stores from an arraycopy being optimized const DecoratorSet C2_ARRAY_COPY = DECORATOR_LAST << 10; diff -r 0a9cb99a98a0 -r 744dc9c33676 src/hotspot/share/gc/shared/c2/modRefBarrierSetC2.cpp --- a/src/hotspot/share/gc/shared/c2/modRefBarrierSetC2.cpp Mon Mar 11 00:06:48 2019 +0100 +++ b/src/hotspot/share/gc/shared/c2/modRefBarrierSetC2.cpp Mon Mar 11 02:05:07 2019 -0400 @@ -1,5 +1,5 @@ /* - * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it @@ -40,9 +40,9 @@ bool anonymous = (decorators & ON_UNKNOWN_OOP_REF) != 0; bool in_heap = (decorators & IN_HEAP) != 0; bool use_precise = is_array || anonymous; - bool tighly_coupled_alloc = (decorators & C2_TIGHLY_COUPLED_ALLOC) != 0; + bool tightly_coupled_alloc = (decorators & C2_TIGHTLY_COUPLED_ALLOC) != 0; - if (!access.is_oop() || tighly_coupled_alloc || (!in_heap && !anonymous)) { + if (!access.is_oop() || tightly_coupled_alloc || (!in_heap && !anonymous)) { return BarrierSetC2::store_at_resolved(access, val); } diff -r 0a9cb99a98a0 -r 744dc9c33676 src/hotspot/share/gc/shenandoah/c2/shenandoahBarrierSetC2.cpp --- a/src/hotspot/share/gc/shenandoah/c2/shenandoahBarrierSetC2.cpp Mon Mar 11 00:06:48 2019 +0100 +++ b/src/hotspot/share/gc/shenandoah/c2/shenandoahBarrierSetC2.cpp Mon Mar 11 02:05:07 2019 -0400 @@ -657,7 +657,7 @@ static_cast(val.type()), NULL /* pre_val */, access.type()); } else { assert(access.is_opt_access(), "only for optimization passes"); - assert(((decorators & C2_TIGHLY_COUPLED_ALLOC) != 0 || !ShenandoahSATBBarrier) && (decorators & C2_ARRAY_COPY) != 0, "unexpected caller of this code"); + assert(((decorators & C2_TIGHTLY_COUPLED_ALLOC) != 0 || !ShenandoahSATBBarrier) && (decorators & C2_ARRAY_COPY) != 0, "unexpected caller of this code"); C2OptAccess& opt_access = static_cast(access); PhaseGVN& gvn = opt_access.gvn(); MergeMemNode* mm = opt_access.mem(); diff -r 0a9cb99a98a0 -r 744dc9c33676 src/hotspot/share/opto/arraycopynode.cpp --- a/src/hotspot/share/opto/arraycopynode.cpp Mon Mar 11 00:06:48 2019 +0100 +++ b/src/hotspot/share/opto/arraycopynode.cpp Mon Mar 11 02:05:07 2019 -0400 @@ -1,5 +1,5 @@ /* - * Copyright (c) 2016, 2018, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2016, 2019, Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it @@ -160,7 +160,7 @@ void ArrayCopyNode::store(BarrierSetC2* bs, PhaseGVN *phase, Node*& ctl, MergeMemNode* mem, Node* adr, const TypePtr* adr_type, Node* val, const Type *type, BasicType bt) { DecoratorSet decorators = C2_WRITE_ACCESS | IN_HEAP | C2_ARRAY_COPY; if (is_alloc_tightly_coupled()) { - decorators |= C2_TIGHLY_COUPLED_ALLOC; + decorators |= C2_TIGHTLY_COUPLED_ALLOC; } C2AccessValuePtr addr(adr, adr_type); C2AccessValue value(val, type); diff -r 0a9cb99a98a0 -r 744dc9c33676 src/hotspot/share/opto/arraycopynode.hpp --- a/src/hotspot/share/opto/arraycopynode.hpp Mon Mar 11 00:06:48 2019 +0100 +++ b/src/hotspot/share/opto/arraycopynode.hpp Mon Mar 11 02:05:07 2019 -0400 @@ -47,7 +47,7 @@ static const char* _kind_names[CopyOfRange+1]; #endif // Is the alloc obtained with - // AllocateArrayNode::Ideal_array_allocation() tighly coupled + // AllocateArrayNode::Ideal_array_allocation() tightly coupled // (arraycopy follows immediately the allocation)? // We cache the result of LibraryCallKit::tightly_coupled_allocation // here because it's much easier to find whether there's a tightly diff -r 0a9cb99a98a0 -r 744dc9c33676 src/hotspot/share/opto/library_call.cpp --- a/src/hotspot/share/opto/library_call.cpp Mon Mar 11 00:06:48 2019 +0100 +++ b/src/hotspot/share/opto/library_call.cpp Mon Mar 11 02:05:07 2019 -0400 @@ -4461,7 +4461,7 @@ return true; } -// If we have a tighly coupled allocation, the arraycopy may take care +// If we have a tightly coupled allocation, the arraycopy may take care // of the array initialization. If one of the guards we insert between // the allocation and the arraycopy causes a deoptimization, an // unitialized array will escape the compiled method. To prevent that