jhuber6 added a comment. In D141051#4048456 <https://reviews.llvm.org/D141051#4048456>, @srj wrote:
> For reasons that aren't yet clear to me, this change is failing to compile > when using gcc-7 and targeting 32-bit targets; the error is of the form > > AMDGPU.cpp:773:10: error: could not convert ‘GPUArchs’ from > ‘llvm::SmallVector<std::__cxx11::basic_string<char>, 1>’ to > ‘llvm::Expected<llvm::SmallVector<std::__cxx11::basic_string<char> > >’ > return GPUArchs; Probably the older GCC doesn't handle the implicit copy elision to the expected type well and thinks that it's copied. I'll put an explicit move on it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141051/new/ https://reviews.llvm.org/D141051 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
