llvm dev - May 2014 - [LLVMdev] [AArch64] Remaining broken tests

Hi Tim,

There are three remaining broken tests on AArch64 on Gabor's buildbot:

http://lab.llvm.org:8011/builders/llvm-aarch64-linux

CodeGen/Generic/2011-07-07-ScheduleDAGCrash.ll
 ** LLVM ERROR: Can't handle live physical register dependency!

CodeGen/Generic/asm-large-immediate.ll
 ** error: invalid operand in inline asm: '/* result: ${0:c} */'

Transforms/BranchFolding/2007-10-19-InlineAsmDirectives.ll
 ** error: invalid operand in inline asm: '.foo_directive ${0:c}:${1:c}'

Also, Gabor, there is one left-over assembly file that must be manually removed:

CodeGen/ARM64/indexed-vector-ldst-2.s

cheers,
--renato

> There are three remaining broken tests on AArch64 on Gabor's buildbot:
More worryingly, we can't self-host. I think I know what the issue is
(http://llvm.org/PR19850), and should have a fix soonish.
> http://lab.llvm.org:8011/builders/llvm-aarch64-linux
>
> CodeGen/Generic/2011-07-07-ScheduleDAGCrash.ll
>  ** LLVM ERROR: Can't handle live physical register dependency!
I've not looked into this one yet. Wide types seem to be involved, but
beyond that I'm not sure.
> CodeGen/Generic/asm-large-immediate.ll
>  ** error: invalid operand in inline asm: '/* result: ${0:c} */'
>
> Transforms/BranchFolding/2007-10-19-InlineAsmDirectives.ll
>  ** error: invalid operand in inline asm: '.foo_directive
${0:c}:${1:c}'
These seem to be because  we don't support a 'c' operand modifier.
Since GCC doesn't even document it as far as I can see, I'm not
massively concerned.
> Also, Gabor, there is one left-over assembly file that must be manually
removed:
>
> CodeGen/ARM64/indexed-vector-ldst-2.s
Yeah, sorry about that. I suspect some mistake in the first commit
that left a stray file in the source directory.

Cheers.

Tim.

On 25 May 2014 20:57, Tim Northover <t.p.northover at gmail.com>
wrote:
> These seem to be because  we don't support a 'c' operand
modifier.
> Since GCC doesn't even document it as far as I can see, I'm not
> massively concerned.
Should we disable them?

--renato