RFR: 8263164: assert(_base >= VectorA && _base <= VectorZ) failed: Not a Vector while calling StoreVectorNode::memory_size() [v4]

[Jie Fu]

On Fri, 12 Mar 2021 11:40:53 GMT, Jie Fu <jiefu at openjdk.org> wrote:

> Why not simply cache the vector type of value being stored during `StoreVectorNode` construction and use it for subsequent queries? Most of the vector nodes already do that since `VectorNode` subclasses `TypeNode`, but `StoreVectorNode` is a `MemNode`.
> 
> And that's the reason I don't understand why non-`StoreVectorNode`-related changes are needed in your patch.
> 
> Nevertheless, there's another case that seems to suffer from the very same problem - `ReductionNode`.

Hi @iwanowww ,

`_vect_type` has been added into StoreVectorNode to cache the vector type of value being stored during StoreVectorNode construction.

I didn't get your point why ReductionNode also suffers from the very same problem.
Could you please make it more clearer?
Thanks.

Best regards,
Jie

-------------

PR: https://git.openjdk.java.net/jdk/pull/2867