[PATCH] D98552: [NFC] Adjust SmallVector.h header to workaround XL build compiler issue

Xiangling Liao via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 12 13:37:34 PST 2021


Xiangling_L created this revision.
Xiangling_L added reviewers: rzurob, hubert.reinterpretcast, jsji, lei, cebowleratibm.
Herald added a subscriber: dexonsmith.
Xiangling_L requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

Previously we delivered a patch to workaround the building issue related to SmallVector here:
https://reviews.llvm.org/D98265.

But in order to prevent further building issues related to the usage of it in other compilation unit, this patch adjusts the `llvm.h` header as a workaround instead.

We will further investigate the real problem of build compiler as a follow-up.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D98552

Files:
  clang/include/clang/Basic/LLVM.h


Index: clang/include/clang/Basic/LLVM.h
===================================================================
--- clang/include/clang/Basic/LLVM.h
+++ clang/include/clang/Basic/LLVM.h
@@ -22,6 +22,8 @@
 // None.h includes an enumerator that is desired & cannot be forward declared
 // without a definition of NoneType.
 #include "llvm/ADT/None.h"
+// Add this header as a workaround to unblock the XL build compiler issue.
+#include "llvm/ADT/SmallVector.h"
 
 namespace llvm {
   // ADT's.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D98552.330357.patch
Type: text/x-patch
Size: 489 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210312/90ea2b42/attachment.bin>


More information about the cfe-commits mailing list