[PATCH] Add ; to acceptable chars to leave unquoted

Rafael EspĂ­ndola rafael.espindola at gmail.com
Sat May 18 09:58:14 PDT 2013


needs a testcase at least.

On 17 May 2013 20:36, Matt Arsenault <Matthew.Arsenault at amd.com> wrote:
> AMDIL uses ; in PrivateGlobalPrefix, and none of the targets' tests seem to mind. Without this, the names get incorrectly quoted.
>
> http://llvm-reviews.chandlerc.com/D816
>
> Files:
>   lib/MC/MCSymbol.cpp
>
> Index: lib/MC/MCSymbol.cpp
> ===================================================================
> --- lib/MC/MCSymbol.cpp
> +++ lib/MC/MCSymbol.cpp
> @@ -21,7 +21,7 @@
>    if ((C < 'a' || C > 'z') &&
>        (C < 'A' || C > 'Z') &&
>        (C < '0' || C > '9') &&
> -      C != '_' && C != '$' && C != '.' && C != '@')
> +      C != '_' && C != '$' && C != '.' && C != '@' && C != ';')
>      return false;
>    return true;
>  }
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>



More information about the llvm-commits mailing list