<div dir='auto'><div>Right. I guess with an old enough bison version, this can happen. I'll see if I can come up with something better.<br><div class="gmail_extra"><br><div class="gmail_quote">On May 21, 2019 15:47, Brian Paul <brianp@vmware.com> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">I think this change broke the MSVC build for us.
<br>

<br>
I may not have time to investigate until later today.
<br>

<br>
-Brian
<br>

<br>

<br>
On 05/21/2019 05:41 AM, GitLab Mirror wrote:
<br>
> Module: Mesa
<br>
> Branch: master
<br>
> Commit: eb85124a9f6e9cb94d0d4a99f91bbae374777e3a
<br>
> URL:    https://nam04.safelinks.protection.outlook.com/?url=http%3A%2F%2Fcgit.freedesktop.org%2Fmesa%2Fmesa%2Fcommit%2F%3Fid%3Deb85124a9f6e9cb94d0d4a99f91bbae374777e3a&amp;data=02%7C01%7Cbrianp%40vmware.com%7C8fb4b04aba0f46cf005d08d6dde15558%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C636940357210597924&amp;sdata=N0sBC6rz%2F4KcpZyCKuNbNqNNhSE%2Fu6d8DcMCQY7STkY%3D&amp;reserved=0
<br>
> 
<br>
> Author: Erik Faye-Lund <erik.faye-lund@collabora.com>
<br>
> Date:   Mon May 20 13:29:05 2019 +0200
<br>
> 
<br>
> glsl: do not use deprecated bison-keyword
<br>
> 
<br>
> %error-verbose has been deprecated since Bison 3.0, which was released
<br>
> in 2013. In Bison 3.3.1 which was recently released, this has started
<br>
> causing warnings. Let's update the code to do this in the modern way
<br>
> intead, to avoid cluttering the output needlessly.
<br>
> 
<br>
> Signed-off-by: Erik Faye-Lund <erik.faye-lund@collabora.com>
<br>
> Reviewed-by: Timothy Arceri <tarceri@itsqueeze.com>
<br>
> 
<br>
> ---
<br>
> 
<br>
>   src/compiler/glsl/glcpp/glcpp-parse.y | 2 +-
<br>
>   src/compiler/glsl/glsl_parser.yy      | 2 +-
<br>
>   src/mesa/program/program_parse.y      | 2 +-
<br>
>   3 files changed, 3 insertions(+), 3 deletions(-)
<br>
> 
<br>
> diff --git a/src/compiler/glsl/glcpp/glcpp-parse.y b/src/compiler/glsl/glcpp/glcpp-parse.y
<br>
> index 1c095cb66f9..736af7e680d 100644
<br>
> --- a/src/compiler/glsl/glcpp/glcpp-parse.y
<br>
> +++ b/src/compiler/glsl/glcpp/glcpp-parse.y
<br>
> @@ -155,7 +155,7 @@ add_builtin_define(glcpp_parser_t *parser, const char *name, int value);
<br>
>   %}
<br>
>   
<br>
>   %pure-parser
<br>
> -%error-verbose
<br>
> +%define parse.error verbose
<br>
>   
<br>
>   %locations
<br>
>   %initial-action {
<br>
> diff --git a/src/compiler/glsl/glsl_parser.yy b/src/compiler/glsl/glsl_parser.yy
<br>
> index 6426f890b9e..dc6aade2643 100644
<br>
> --- a/src/compiler/glsl/glsl_parser.yy
<br>
> +++ b/src/compiler/glsl/glsl_parser.yy
<br>
> @@ -81,7 +81,7 @@ static bool match_layout_qualifier(const char *s1, const char *s2,
<br>
>   %expect 0
<br>
>   
<br>
>   %pure-parser
<br>
> -%error-verbose
<br>
> +%define parse.error verbose
<br>
>   
<br>
>   %locations
<br>
>   %initial-action {
<br>
> diff --git a/src/mesa/program/program_parse.y b/src/mesa/program/program_parse.y
<br>
> index 7398f5f507a..3d0c1e2ea9e 100644
<br>
> --- a/src/mesa/program/program_parse.y
<br>
> +++ b/src/mesa/program/program_parse.y
<br>
> @@ -124,7 +124,7 @@ static struct asm_instruction *asm_instruction_copy_ctor(
<br>
>   %locations
<br>
>   %lex-param   { struct asm_parser_state *state }
<br>
>   %parse-param { struct asm_parser_state *state }
<br>
> -%error-verbose
<br>
> +%define parse.error verbose
<br>
>   
<br>
>   %union {
<br>
>      struct asm_instruction *inst;
<br>
> 
<br>
> _______________________________________________
<br>
> mesa-commit mailing list
<br>
> mesa-commit@lists.freedesktop.org
<br>
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Fmesa-commit&amp;data=02%7C01%7Cbrianp%40vmware.com%7C8fb4b04aba0f46cf005d08d6dde15558%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C636940357210607919&amp;sdata=%2F%2FK4CMY1Wd9PgydEgugYq63pp26NkX%2B4venu%2FYk7FQk%3D&amp;reserved=0
<br>
> 
<br>

<br>
</p>
</blockquote></div><br></div></div></div>