I agree. For PlatformMacOSX, we care to make sure no vendor was specified
(unspecified unknown) or we want MacOSX, iOS, or Darwin and nothing else. For
Linux, just the OS should be enough.
Feel free to remove the vendor test as this was probably copied from one of the
Mac or iOS platforms.
Greg Clayton
> On Mar 26, 2015, at 9:27 AM, Ted Woodward <ted.woodw...@codeaurora.org> wrote:
>
> PlatformLinux::CreateInstance() has this code, which checks the Vendor in
> the Triple:
>
> switch (triple.getVendor())
> {
> case llvm::Triple::PC:
> create = true;
> break;
>
> #if defined(__linux__)
> // Only accept "unknown" for the vendor if the host is linux and
> // it "unknown" wasn't specified (it was just returned because
> it
> // was NOT specified_
> case llvm::Triple::VendorType::UnknownVendor:
> create = !arch->TripleVendorWasSpecified();
> break;
> #endif
> default:
> break;
> }
>
>
> I don't think the Linux platform should care about the Vendor, just the Core
> and OS. What do you guys think?
>
> --
> Qualcomm Innovation Center, Inc.
> The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a
> Linux Foundation Collaborative Project
>
>
> _______________________________________________
> lldb-dev mailing list
> lldb-dev@cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
_______________________________________________
lldb-dev mailing list
lldb-dev@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
