[llvm] r330453 - [utils] improve AArch64 asm parser

Fri Apr 20 11:45:20 PDT 2018

Thanks, that did it!

On Fri, Apr 20, 2018 at 8:16 PM, Sanjay Patel via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> Author: spatel
> Date: Fri Apr 20 10:16:23 2018
> New Revision: 330453
>
> URL: http://llvm.org/viewvc/llvm-project?rev=330453&view=rev
> Log:
> [utils] improve AArch64 asm parser
>
> If we don't mark the cfi line as optional, the script won't
> work with 'nounwind' code. Without that attr, there may be
> extra noise in the asm body that we don't want to see.
>
> Modified:
>     llvm/trunk/utils/UpdateTestChecks/asm.py
>
> Modified: llvm/trunk/utils/UpdateTestChecks/asm.py
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/UpdateTestChecks/asm.py?rev=330453&r1=330452&r2=330453&view=diff
> ==============================================================================
> --- llvm/trunk/utils/UpdateTestChecks/asm.py (original)
> +++ llvm/trunk/utils/UpdateTestChecks/asm.py Fri Apr 20 10:16:23 2018
> @@ -28,7 +28,7 @@ ASM_FUNCTION_ARM_RE = re.compile(
>
>  ASM_FUNCTION_AARCH64_RE = re.compile(
>       r'^_?(?P<func>[^:]+):[ \t]*\/\/[ \t]*@(?P=func)\n'
> -     r'[ \t]+.cfi_startproc\n'
> +     r'(?:[ \t]+.cfi_startproc\n)?'  # drop optional cfi noise
>       r'(?P<body>.*?)\n'
>       # This list is incomplete
>       r'.Lfunc_end[0-9]+:\n',
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits