Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/17627 )

Change subject: [thirdparty] fix building LLVM 9 with GCC 10
......................................................................

[thirdparty] fix building LLVM 9 with GCC 10

This patch fixes LLVM 9 build with GCC 10 in Kudu's thirdparty.  The
imported patch is already in the LLVM repo, see [1].

Prior to this patch, LLVM9 build in thirdparty would fail with the
following error:

  
thirdparty/src/llvm-9.0.0.src/include/llvm/Demangle/MicrosoftDemangleNodes.h:566:3:
 error: unknown type name 'uint32_t'
    uint32_t NVOffset = 0;
    ^

[1] 
https://github.com/llvm-mirror/llvm/commit/e0402b5c9813a2458b8dd3f640883110db280395

Change-Id: I3332556229dbf2bce65d52f37c48d62624465f70
Reviewed-on: http://gerrit.cloudera.org:8080/17627
Tested-by: Kudu Jenkins
Reviewed-by: Andrew Wong <[email protected]>
---
M thirdparty/download-thirdparty.sh
A 
thirdparty/patches/llvm-MicrosoftDemangleNodes-e0402b5c9813a2458b8dd3f640883110db280395.patch
2 files changed, 34 insertions(+), 1 deletion(-)

Approvals:
  Kudu Jenkins: Verified
  Andrew Wong: Looks good to me, approved

-- 
To view, visit http://gerrit.cloudera.org:8080/17627
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I3332556229dbf2bce65d52f37c48d62624465f70
Gerrit-Change-Number: 17627
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to