[PATCH] D115859: [LoongArch 2/6] Add ELF machine flag and relocs for upcoming LoongArch target

[Shubham Sandeep Rastogi via Phabricator via llvm-commits]

rastogishubham added a comment.

Hi it seems like this patch caused an error in the lldb build machines:

/Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/llvm/include/llvm/BinaryFormat/ELF.h:877:1: error: expected identifier
#include "ELFRelocs/LoongArch.def"
^
/Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/llvm/include/llvm/BinaryFormat/ELF.h:877:1: error: expected '}'
/Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/llvm/include/llvm/BinaryFormat/ELF.h:876:6: note: to match this '{'
enum {

  ^

/Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/llvm/include/llvm/BinaryFormat/ELF.h:876:7: error: expected ';' after enum
enum {

  ^
  ;

/Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/llvm/include/llvm/BinaryFormat/ELF.h:876:1: warning: declaration does not declare anything [-Wmissing-declarations]
enum {
^
/Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/llvm/include/llvm/BinaryFormat/ELF.h:877:1: fatal error: import of module 'LLVM_BinaryFormat.ELFRelocs.LoongArch' appears within namespace 'llvm::ELF'
#include "ELFRelocs/LoongArch.def"
^
/Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/llvm/include/llvm/BinaryFormat/ELF.h:27:1: note: namespace 'llvm::ELF' begins here
namespace ELF {
^
/Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/llvm/lib/BinaryFormat/AMDGPUMetadataVerifier.cpp:14:10: fatal error: could not build module 'LLVM_BinaryFormat'
#include "llvm/BinaryFormat/AMDGPUMetadataVerifier.h"
 ~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

link to bot here: https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/41227/

Please fix or revert your patch as soon as possible!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D115859/new/

https://reviews.llvm.org/D115859