[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