Just went in: Sending include/lldb/Core/ArchSpec.h Sending source/Core/ArchSpec.cpp Transmitting file data .. Committed revision 212145.
On Tue, Jul 1, 2014 at 4:34 PM, Greg Clayton <[email protected]> wrote: > Looks good. > > > On Jul 1, 2014, at 2:45 PM, Todd Fiala <[email protected]> wrote: > > > > LGTM from my end. If Greg is okay with it, I'll check it in. > > > > Also: > > No build issues on Ubuntu 14.04 or MacOSX. > > No new test failures on Ubuntu 14.04. > > > > -Todd > > > > > > On Tue, Jul 1, 2014 at 11:09 AM, Todd Fiala <[email protected]> wrote: > > I'll get them in my queue and run the tests. (Couple things ahead of > it). > > > > > > On Tue, Jul 1, 2014 at 2:35 AM, Matthew Gardiner <[email protected]> wrote: > > Hi folks > > > > Would someone be able to submit this patch for me please? I have been > trying over the past few months to extend lldb to debug CSRs Kalimba DSPs. > I could really do with getting at least these changes upstream before I > start pushing more features. > > > > Index: include/lldb/Core/ArchSpec.h > > =================================================================== > > --- include/lldb/Core/ArchSpec.h (revision 212092) > > +++ include/lldb/Core/ArchSpec.h (working copy) > > @@ -101,6 +101,9 @@ > > > > eCore_uknownMach32, > > eCore_uknownMach64, > > + > > + eCore_kalimba, > > + > > kNumCores, > > > > kCore_invalid, > > Index: source/Core/ArchSpec.cpp > > =================================================================== > > --- source/Core/ArchSpec.cpp (revision 212092) > > +++ source/Core/ArchSpec.cpp (working copy) > > @@ -113,7 +113,9 @@ > > { eByteOrderLittle, 4, 4, 4, llvm::Triple::hexagon , > ArchSpec::eCore_hexagon_hexagonv5, "hexagonv5" }, > > > > { eByteOrderLittle, 4, 4, 4 , llvm::Triple::UnknownArch , > ArchSpec::eCore_uknownMach32 , "unknown-mach-32" }, > > - { eByteOrderLittle, 8, 4, 4 , llvm::Triple::UnknownArch , > ArchSpec::eCore_uknownMach64 , "unknown-mach-64" } > > + { eByteOrderLittle, 8, 4, 4 , llvm::Triple::UnknownArch , > ArchSpec::eCore_uknownMach64 , "unknown-mach-64" }, > > + > > + { eByteOrderLittle, 4, 1, 1 , llvm::Triple::UnknownArch , > ArchSpec::eCore_kalimba , "kalimba" } > > }; > > > > struct ArchDefinitionEntry > > @@ -249,7 +251,9 @@ > > { ArchSpec::eCore_sparc9_generic , llvm::ELF::EM_SPARCV9, > LLDB_INVALID_CPUTYPE, 0xFFFFFFFFu, 0xFFFFFFFFu }, // SPARC V9 > > { ArchSpec::eCore_x86_64_x86_64 , llvm::ELF::EM_X86_64 , > LLDB_INVALID_CPUTYPE, 0xFFFFFFFFu, 0xFFFFFFFFu }, // AMD64 > > { ArchSpec::eCore_mips64 , llvm::ELF::EM_MIPS , > LLDB_INVALID_CPUTYPE, 0xFFFFFFFFu, 0xFFFFFFFFu }, // MIPS > > - { ArchSpec::eCore_hexagon_generic , llvm::ELF::EM_HEXAGON, > LLDB_INVALID_CPUTYPE, 0xFFFFFFFFu, 0xFFFFFFFFu } // HEXAGON > > + { ArchSpec::eCore_hexagon_generic , llvm::ELF::EM_HEXAGON, > LLDB_INVALID_CPUTYPE, 0xFFFFFFFFu, 0xFFFFFFFFu }, // HEXAGON > > + { ArchSpec::eCore_kalimba , llvm::ELF::EM_CSR_KALIMBA, > LLDB_INVALID_CPUTYPE, 0xFFFFFFFFu, 0xFFFFFFFFu } // KALIMBA > > + > > }; > > > > static const ArchDefinition g_elf_arch_def = { > > > > > > Please note that I have left the field llvm::Triple::ArchType as > "UnknownArch" for now, as I'm unsure as to how to proceed with changes to > the Triple.h/.cpp stuff as yet. > > > > Patch file attached. > > > > thanks, > > Matthew Gardiner > > > > > > Member of the CSR plc group of companies. CSR plc registered in England > and Wales, registered number 4187346, registered office Churchill House, > Cambridge Business Park, Cowley Road, Cambridge, CB4 0WZ, United Kingdom > > More information can be found at www.csr.com. Keep up to date with CSR > on our technical blog, www.csr.com/blog, CSR people blog, > www.csr.com/people, YouTube, www.youtube.com/user/CSRplc, Facebook, > www.facebook.com/pages/CSR/191038434253534, or follow us on Twitter at > www.twitter.com/CSR_plc. > > New for 2014, you can now access the wide range of products powered by > aptX at www.aptx.com. > > > > _______________________________________________ > > lldb-commits mailing list > > [email protected] > > http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits > > > > > > > > > > -- > > Todd Fiala | Software Engineer | [email protected] | > 650-943-3180 > > > > > > _______________________________________________ > > lldb-commits mailing list > > [email protected] > > http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits > > > > > > > > > > -- > > -Todd > > _______________________________________________ > > lldb-commits mailing list > > [email protected] > > http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits > > -- -Todd
_______________________________________________ lldb-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
