RFR: 8310308: IR Framework: check for type and size of vector nodes [v2]
Emanuel Peter
epeter at openjdk.org
Fri Jun 30 09:29:05 UTC 2023
On Wed, 28 Jun 2023 12:35:16 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:
>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>>
>> revert to ANY for TestAutoVectorization2DArray.java
>
> test/hotspot/jtreg/compiler/lib/ir_framework/IRNode.java line 2250:
>
>> 2248: int s = 0;
>> 2249: try {
>> 2250: s = Integer.parseInt(sizes[i]);
>
> We should also check if the size is a reasonable number (i.e. a positive multiple of 2 and maybe an upper limit(?)) and report a format violation if that is not the case as for example in:
>
> @IR(counts = {IRNode.ADD_VI, IRNode.VECTOR_SIZE + "3,-2", "1"})
I'm reporting an error on negative numbers now. But I'm not sure I want to limit it to power of 2 or a size limit. Especially when uses flags/tags this could not always hold.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14539#discussion_r1247656570
More information about the hotspot-compiler-dev
mailing list