Re: [PATCH v2 9/9] RISC-V: Add cost model asserts

2024-08-27 Thread Patrick O'Neill
On 8/27/24 08:19, Jeff Law wrote: On 8/26/24 6:37 PM, Patrick O'Neill wrote: This patch adds some advanced checking to assert that the emitted costs match emitted patterns for const_vecs. Flow: Costing: Insert into hashmap> Expand: Check for membership in hashmap   -> Not in hashmap: ignor

Re: [PATCH v2 9/9] RISC-V: Add cost model asserts

2024-08-27 Thread Jeff Law
On 8/26/24 6:37 PM, Patrick O'Neill wrote: This patch adds some advanced checking to assert that the emitted costs match emitted patterns for const_vecs. Flow: Costing: Insert into hashmap> Expand: Check for membership in hashmap -> Not in hashmap: ignore, this wasn't costed -> In hashmap

[PATCH v2 9/9] RISC-V: Add cost model asserts

2024-08-26 Thread Patrick O'Neill
This patch adds some advanced checking to assert that the emitted costs match emitted patterns for const_vecs. Flow: Costing: Insert into hashmap> Expand: Check for membership in hashmap -> Not in hashmap: ignore, this wasn't costed -> In hashmap: Iterate over vec -> if RTX not in hashmap: I