[Mesa-dev] [PATCH] docs: Fix GLSL compiler link

[Alejandro Piñeiro]

On 12/01/17 15:09, Elie Tournier wrote:
> The doc wasn't update since we moved the glsl compiler to src/compiler/glsl.
> I also updated the description of the standalone compiler.
>
> Signed-off-by: Elie Tournier <tournier.elie at gmail.com>
> ---
>  docs/shading.html | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/docs/shading.html b/docs/shading.html
> index cf989ce902..f50f316f2d 100644
> --- a/docs/shading.html
> +++ b/docs/shading.html
> @@ -180,7 +180,7 @@ Here's an example of using the compiler to compile a vertex shader and
>  emit GL_ARB_vertex_program-style instructions:
>  </p>
>  <pre>
> -    src/glsl/glsl_compiler --dump-ast myshader.vert
> +    src/compiler/glsl/glsl_compiler --dump-ast myshader.vert
>  </pre>
>  
>  Options include
> @@ -188,7 +188,10 @@ Options include
>  <li><b>--dump-ast</b> - dump GPU code
>  <li><b>--dump-hir</b> - dump high-level IR code
>  <li><b>--dump-lir</b> - dump low-level IR code
> -<li><b>--link</b> - ???
> +<li><b>--dump-builder</b> - dump GLSL IR code
> +<li><b>--link</b> - link shaders
> +<li><b>--just-log</b> - display shader / linker info if exist

Here I would mention that removes headers/separators, because if not it
is not clear the difference between using the option or not. Something like:
"display only shader / linker info if exist, without any header or
separator"

Feel free to tweak the wording.

> +<li><b>--version</b> - define the GLSL version to use

It would be good to mention that this argument is mandatory.

>  </ul>
>  
>  
> @@ -196,7 +199,7 @@ Options include
>  
>  <p>
>  The source code for Mesa's shading language compiler is in the
> -<code>src/glsl/</code> directory.
> +<code>src/compiler/glsl/</code> directory.
>  </p>
>  
>  <p>

With those changes:
Reviewed-by: Alejandro Piñeiro <apinheiro at igalia.com>