[llvm] r192352 - Implement AArch64 vector load/store multiple N-element structure class SIMD(lselem).

Hao Liu Hao.Liu at arm.com
Thu Oct 10 09:49:14 PDT 2013


Hi Rafael and Tim,

That's an interesting problem. 
I used g++ to compile, there is no build failure. It only happens when using
clang++.

I've found the failure is caused by missing declarations . 
In AArch64Disassembler.cpp, the code is like following:
(1)  DecodeDPairRegisterClass(...);
      ... 
(2)  #include "AArch64GenDisassemblerTable.inc"
      ... 
(3)  DecodeDPairRegisterClass(...) {
          ...
      }

My patch don't have the declare for decode function (1). 
So in "AArch64GenDisassemblerTable.inc",  it can't find
DecodeDPairRegisterClass. And the build is failed. 
Now I've added the declarations, and there is no build failures.

If it's OK, I'll recommit the patch.

Thanks,
-Hao

-----Original Message-----
From: Tim Northover [mailto:t.p.northover at gmail.com] 
Sent: Thursday, October 10, 2013 5:06 PM
To: Hao Liu
Cc: llvm-commits; Rafael EspĂ­ndola
Subject: Re: [llvm] r192352 - Implement AArch64 vector load/store multiple
N-element structure class SIMD(lselem).

On 10 October 2013 16:55, Rafael EspĂ­ndola <rafael.espindola at gmail.com>
wrote:
> I reverted this since it broke the build.

And I've done the same to the Clang side since the test would fail.
I'm sure your mailbox is overflowing with buildbot messages, but just in
case they got lost, here's an example of the failure:

http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-debian-fast/builds/8
064

Some name lookup thing that it looks like GCC doesn't know about.

Cheers.

Tim.







More information about the llvm-commits mailing list