diff options
| author | Jonas Devlieghere <jonas@devlieghere.com> | 2020-10-23 09:26:11 -0700 |
|---|---|---|
| committer | Jonas Devlieghere <jonas@devlieghere.com> | 2020-10-23 10:00:23 -0700 |
| commit | de346cf2ac850be793ee7fc31f99a7671331c289 (patch) | |
| tree | 051ba1beecff0cc9a7672840b0e79d629a801703 /clang/lib/Basic/SourceManager.cpp | |
| parent | 9df832d1c3aa014c88ec947df3677da2e7d8bd64 (diff) | |
| download | llvm-de346cf2ac850be793ee7fc31f99a7671331c289.zip llvm-de346cf2ac850be793ee7fc31f99a7671331c289.tar.gz llvm-de346cf2ac850be793ee7fc31f99a7671331c289.tar.bz2 | |
[lldb] Redesign Target::GetUtilityFunctionForLanguage API
This patch redesigns the Target::GetUtilityFunctionForLanguage API:
- Use a unique_ptr instead of a raw pointer for the return type.
- Wrap the result in an llvm::Expected instead of using a Status object as an I/O parameter.
- Combine the action of "getting" and "installing" the UtilityFunction as they always get called together.
- Pass std::strings instead of const char* and std::move them where appropriate.
There's more room for improvement but I think this tackles the most
prevalent issues with the current API.
Differential revision: https://reviews.llvm.org/D90011
Diffstat (limited to 'clang/lib/Basic/SourceManager.cpp')
0 files changed, 0 insertions, 0 deletions