[Lldb-commits] [lldb] [lldb] Add AMD and NVIDIA GPU architectures to ArchSpec (PR #206246)

[satyanarayana reddy janga via lldb-commits]

================
@@ -618,20 +646,13 @@ ModuleSpecList ObjectFileELF::GetModuleSpecifications(
 
       if (spec.GetArchitecture().IsValid()) {
         llvm::Triple::OSType ostype;
-        llvm::Triple::VendorType vendor;
         llvm::Triple::OSType spec_ostype =
             spec.GetArchitecture().GetTriple().getOS();
 
         LLDB_LOGF(log, "ObjectFileELF::%s file '%s' module OSABI: %s",
                   __FUNCTION__, file.GetPath().c_str(),
                   OSABIAsCString(header.e_ident[EI_OSABI]));
 
-        // SetArchitecture should have set the vendor to unknown
-        vendor = spec.GetArchitecture().GetTriple().getVendor();
-        assert(vendor == llvm::Triple::UnknownVendor);
----------------
satyajanga wrote:

AMD triple sets the vendor to `llvm::Triple::AMD` .  so the invariant set by the variant is not longer true. 

https://github.com/llvm/llvm-project/pull/206246