[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 15:13:26 PST 2021


Xiangling_L added inline comments.


================
Comment at: clang/include/clang/Basic/LLVM.h:71
   using llvm::SmallString;
-  using llvm::SmallVector;
-  using llvm::SmallVectorImpl;
----------------
It's my bad to not wait for a build finished before updating the patch, I saw a lot of build failures as the following that seems related to the deletion here:
```
/data/xling/wyvern/master-external/llvm-project/clang/include/clang/Basic/Module.h:222:3: error: no template named 'SmallVector'; did you mean 'llvm::SmallVector'?
  SmallVector<UnresolvedHeaderDirective, 1> UnresolvedHeaders;
  ^~~~~~~~~~~
  llvm::SmallVector
/data/xling/wyvern/master-external/llvm-project/clang/include/clang/Basic/LLVM.h:38:42: note: 'llvm::SmallVector' declared here
  template<typename T, unsigned N> class SmallVector;
                                         ^
```
I will update the patch accordingly after I have a successful build and run the test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98552



More information about the cfe-commits mailing list