On Mon, Jan 21, 2019 at 10:36:20AM +0100, Martin Liška wrote:
> @@ -11338,7 +11339,7 @@ gfc_match_gcc_unroll (void)
> return MATCH_ERROR;
> }
>
> -/* Match a !GCC$ builtin (b) attributes simd flags form:
> +/* Match a !GCC$ builtin (b) attributes simd if(target) flags form:
>
> The parameter b is name of a middle-end built-in.
> Flags are one of:
> @@ -11346,11 +11347,16 @@ gfc_match_gcc_unroll (void)
> - inbranch
> - notinbranch
>
> + Target must be one of:
> + - lp64
> + - ilp32
> +
Not exactly correct. if(target) is optional, not mandatory, and, if it
appears, must appear after flags. flags is also incorrect, because it is
either nothing, or (inbranch), or (notinbranch), while the function comment
suggests that it is inbranch or notinbranch.
Jakub
