Re: [Mesa-dev] [PATCH 1/8] glsl: calculate number of operands in an expression once

On 09/08/17 05:59, Thomas Helland wrote: I'm not quite sure if the increase in size of the base ir class versus the reduced overhead and added code is worth it? There is more code yes, but most of that is asserts and validation code. This should actually make the code more robust than it is

Re: [Mesa-dev] [PATCH 1/8] glsl: calculate number of operands in an expression once

I'm not quite sure if the increase in size of the base ir class versus the reduced overhead and added code is worth it? Some numbers would be nice. A comment below 2017-08-07 2:18 GMT+00:00 Timothy Arceri : > Extra validation is added to ir_validate to make sure this is >