================
@@ -769,19 +771,20 @@ CompilerType::GetBasicTypeFromAST(lldb::BasicType
basic_type) const {
}
// Exploring the type
-std::optional<uint64_t>
+llvm::Expected<uint64_t>
CompilerType::GetBitSize(ExecutionContextScope *exe_scope) const {
if (IsValid())
if (auto type_system_sp = GetTypeSystem())
return type_system_sp->GetBitSize(m_type, exe_scope);
- return {};
+ return llvm::createStringError("invalid type; cannot determine size");
----------------
JDevlieghere wrote:
```suggestion
return llvm::createStringError("invalid type: cannot determine size");
```
We generally concatenate errors with `:` rather than `;`. Will require updating
the two tests below.
https://github.com/llvm/llvm-project/pull/129601
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
