Hi Vlad,
Welcome to Mesa.
Have you don't any benchmarking on this patch via something like shader-db [1]?
Mentioning the results in the commit message is encouraged.
On 2 June 2017 at 13:37, Vlad Golovkin <vlad.golovkin.m...@gmail.com> wrote:
> This will make it possible to remove unnecessary strlen and strncmp
> calls.
> ---
> src/compiler/glsl/glcpp/glcpp-parse.y | 10 ++++++----
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/src/compiler/glsl/glcpp/glcpp-parse.y
> b/src/compiler/glsl/glcpp/glcpp-parse.y
> index fe211a0f0b..8cdd4c9d5b 100644
> --- a/src/compiler/glsl/glcpp/glcpp-parse.y
> +++ b/src/compiler/glsl/glcpp/glcpp-parse.y
> @@ -450,10 +450,12 @@ control_line_error:
>
> integer_constant:
> INTEGER_STRING {
> - if (strlen ($1) >= 3 && strncmp ($1, "0x", 2) == 0) {
> - $$ = strtoll ($1 + 2, NULL, 16);
> - } else if ($1[0] == '0') {
> - $$ = strtoll ($1, NULL, 8);
> + if ($1[0] == '0') {
> + if ($1[1] == 'x' && $1[2] != '\0') {
> + $$ = strtoll ($1 + 2, NULL, 16);
> + } else {
> + $$ = strtoll ($1, NULL, 8);
> + }
> } else {
> $$ = strtoll ($1, NULL, 10);
> }
Food for thought:
Any idea why we'd want to handle any of these details ourselves?
strtoll(..., /*base*/ 0) is smart enough, plus it should be better optimised.
Another possibility is to swap the INTEGER_STRING token for
{DEC,OCT,HEX}... ones each one calling strtoll() with correct base.
-Emil
[1] https://cgit.freedesktop.org/mesa/shader-db/
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
