[Lldb-commits] [PATCH] D120810: [lldb] Remove the global platform list

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Mar 9 07:10:43 PST 2022


labath added inline comments.


================
Comment at: lldb/source/Target/Platform.cpp:1891
+
+PlatformSP PlatformList::GetOrCreate(const ArchSpec &arch,
+                                     ArchSpec *platform_arch_ptr,
----------------
JDevlieghere wrote:
> [Nit/pedantic] We have a few calls to `IsCompatibleArchitecture` in this function. Based on the existing comments you can infer that that's what the second argument is for. Normally I'd suggest an inline comment, but given the that there's a bunch of calls, could we have either have two constants:
> 
> ```
> enum ArchMatch : bool {
>   ExactArchMatch = true,
>   CompatibleArchMatch = false,
> };
> ```
> 
> Or maybe even better, can we change the signature of that function to take an enum with those values? 
I've created a separate patch for that (https://reviews.llvm.org/D121290)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120810



More information about the lldb-commits mailing list