Summary: Optimization in CmpPNode::sub() removed the valid compare instruction because of false positive answer from detect_dominating_control().
Reviewed-by: jrose, sgoldman

Summary: Compressed oops in instances, arrays, and headers. Code contributors are coleenp, phh, never, swamyv
Reviewed-by: jmasa, kamg, acorn, tbell, kvn, rasbold

Summary: Without LoopNode, exit early from loop unswitching and partial peeling
Reviewed-by: kvn, never, sgoldman

Summary: Don't optimize zero length array allocations at compile time.
Reviewed-by: kvn, never

Reviewed-by: kvn, rasbold

Summary: C2 spends > 60% in escape analysis code during test nsk/regression/b4675027.
Reviewed-by: never

Summary: remove old private plugin interface, simplify, rework old plugin to use unchanged Gnu sources
Reviewed-by: kvn, rasbold

Summary: missing check in LoadNode::Ideal() causes infinite generation of a value Phi.
Reviewed-by: jrose, never

Reviewed-by: kvn, jrose

Summary: The assert misses the case CastX2P 'base' for an unsafe field reference
Reviewed-by: never, jrose

Summary: EA gives opportunite to do more aggressive memory optimizations.
Reviewed-by: never, jrose

Summary: Use scalar replacement with EA to remove allocations for objects which do not escape the compiled method.
Reviewed-by: rasbold, never, jrose

6663621: JVM crashes while trying to execute api/java_security/Signature/SignatureTests.html#initSign tests.
Summary: alignment expression with secondary induction variables is sometimes wrong
Reviewed-by: kvn, rasbold

Summary: copy elimination of a constant value results in incorrect execution
Reviewed-by: kvn, sgoldman, rasbold

Summary: split_thru_phi produces top on a non-dead path
Reviewed-by: kvn, rasbold, sgoldman

Summary: ClearArrayNode::Identity should use TypeX instead of TypeInt
Reviewed-by: jrose, kvn, sgoldman

Summary: Compile::has_loops() should be set from inlined methods
Reviewed-by: kvn, rasbold

Summary: Remove lock/unlock MemBar nodes and specify locks in debug info for deoptimization.
Reviewed-by: never

Summary: Current EA code has several problems which have to be fixed.
Reviewed-by: jrose, sgoldman

Summary: Avoid an infinite generation of instance's field values Phi nodes.
Reviewed-by: never

6671807: (Escape Analysis) Add new ideal node to represent the state of a scalarized object at a safepoint
Summary: Values of non-static fields of a scalarized object should be saved in debug info to reallocate the object during deoptimization.
Reviewed-by: never

Summary: C2 crashed in StoreCMNode::Value because n->in(MemNode::OopStore) is 0.
Reviewed-by: rasbold, never

Summary: java constructors should be inlined to be able scalar replace a new object
Reviewed-by: rasbold

Summary: dump full ideal graph takes forever.
Reviewed-by: never, rasbold

Summary: Cloning an allocation will not allow scalar replacement since memory operations could not be associated with one allocation.
Reviewed-by: rasbold

Summary: During split unique types EA could exceed nodes limit and fail the method compilation.
Reviewed-by: rasbold

6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
Summary: In Parse::do_if() 'c' (CmpNode) node may be changed during BoolNode transformation so 'c' may became dead but the node is referenced later in the code.
Reviewed-by: never

Summary: this optimization causes problems (sizes of Load and Store nodes do not match) for objects initialization code and Escape Analysis
Reviewed-by: jrose, never

Summary: float CMove generation add duplicated CMPF if there are more then one Move depending on the condition.
Reviewed-by: jrose, never, rasbold

Summary: generate_arraycopy() does not check the size of allocated array.
Reviewed-by: jrose, never

Summary: CmpP could be optimized out if it compares new allocated objects.
Reviewed-by: jrose, never, rasbold

Summary: set_req_X() puts dependent nodes on IGVN worklist which allows to improve graph and gives more opportunities for EA scalar replacement.
Reviewed-by: jrose, never

Summary: A mergemem node could be not in worklist_store but in should_not_repeat vectorset since it was processed and removed from worklist_store before.
Reviewed-by: jrose, never

Summary: T_ADDRESS size is defined as 'int' size (4 bytes) but C2 use it for raw pointers and as memory type for StoreP and LoadP nodes.
Reviewed-by: jrose

Summary: AdapterHandlerLibrary::get_create_adapter_index() returns incorrect value (-2) when CodeCache is full.
Reviewed-by: sgoldman

Summary: A node is not processed in dump_nodes() if it was visited during processing previous inputs.
Reviewed-by: rasbold

Summary: '* HeapWordSize' is missing in GraphKit::set_output_for_allocation()
Reviewed-by: rasbold, jrose, never

Reviewed-by: kvn, rasbold