<div dir="ltr">Just went in:<div><div>Sending include/lldb/Core/ArchSpec.h</div><div>Sending source/Core/ArchSpec.cpp</div><div>Transmitting file data ..</div><div>Committed revision 212145.</div></div></div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jul 1, 2014 at 4:34 PM, Greg Clayton <span dir="ltr"><<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Looks good.<br>
<div class="HOEnZb"><div class="h5"><br>
> On Jul 1, 2014, at 2:45 PM, Todd Fiala <<a href="mailto:todd.fiala@gmail.com">todd.fiala@gmail.com</a>> wrote:<br>
><br>
> LGTM from my end. If Greg is okay with it, I'll check it in.<br>
><br>
> Also:<br>
> No build issues on Ubuntu 14.04 or MacOSX.<br>
> No new test failures on Ubuntu 14.04.<br>
><br>
> -Todd<br>
><br>
><br>
> On Tue, Jul 1, 2014 at 11:09 AM, Todd Fiala <<a href="mailto:tfiala@google.com">tfiala@google.com</a>> wrote:<br>
> I'll get them in my queue and run the tests. (Couple things ahead of it).<br>
><br>
><br>
> On Tue, Jul 1, 2014 at 2:35 AM, Matthew Gardiner <<a href="mailto:mg11@csr.com">mg11@csr.com</a>> wrote:<br>
> Hi folks<br>
><br>
> 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.<br>
><br>
> Index: include/lldb/Core/ArchSpec.h<br>
> ===================================================================<br>
> --- include/lldb/Core/ArchSpec.h (revision 212092)<br>
> +++ include/lldb/Core/ArchSpec.h (working copy)<br>
> @@ -101,6 +101,9 @@<br>
><br>
> eCore_uknownMach32,<br>
> eCore_uknownMach64,<br>
> +<br>
> + eCore_kalimba,<br>
> +<br>
> kNumCores,<br>
><br>
> kCore_invalid,<br>
> Index: source/Core/ArchSpec.cpp<br>
> ===================================================================<br>
> --- source/Core/ArchSpec.cpp (revision 212092)<br>
> +++ source/Core/ArchSpec.cpp (working copy)<br>
> @@ -113,7 +113,9 @@<br>
> { eByteOrderLittle, 4, 4, 4, llvm::Triple::hexagon , ArchSpec::eCore_hexagon_hexagonv5, "hexagonv5" },<br>
><br>
> { eByteOrderLittle, 4, 4, 4 , llvm::Triple::UnknownArch , ArchSpec::eCore_uknownMach32 , "unknown-mach-32" },<br>
> - { eByteOrderLittle, 8, 4, 4 , llvm::Triple::UnknownArch , ArchSpec::eCore_uknownMach64 , "unknown-mach-64" }<br>
> + { eByteOrderLittle, 8, 4, 4 , llvm::Triple::UnknownArch , ArchSpec::eCore_uknownMach64 , "unknown-mach-64" },<br>
> +<br>
> + { eByteOrderLittle, 4, 1, 1 , llvm::Triple::UnknownArch , ArchSpec::eCore_kalimba , "kalimba" }<br>
> };<br>
><br>
> struct ArchDefinitionEntry<br>
> @@ -249,7 +251,9 @@<br>
> { ArchSpec::eCore_sparc9_generic , llvm::ELF::EM_SPARCV9, LLDB_INVALID_CPUTYPE, 0xFFFFFFFFu, 0xFFFFFFFFu }, // SPARC V9<br>
> { ArchSpec::eCore_x86_64_x86_64 , llvm::ELF::EM_X86_64 , LLDB_INVALID_CPUTYPE, 0xFFFFFFFFu, 0xFFFFFFFFu }, // AMD64<br>
> { ArchSpec::eCore_mips64 , llvm::ELF::EM_MIPS , LLDB_INVALID_CPUTYPE, 0xFFFFFFFFu, 0xFFFFFFFFu }, // MIPS<br>
> - { ArchSpec::eCore_hexagon_generic , llvm::ELF::EM_HEXAGON, LLDB_INVALID_CPUTYPE, 0xFFFFFFFFu, 0xFFFFFFFFu } // HEXAGON<br>
> + { ArchSpec::eCore_hexagon_generic , llvm::ELF::EM_HEXAGON, LLDB_INVALID_CPUTYPE, 0xFFFFFFFFu, 0xFFFFFFFFu }, // HEXAGON<br>
> + { ArchSpec::eCore_kalimba , llvm::ELF::EM_CSR_KALIMBA, LLDB_INVALID_CPUTYPE, 0xFFFFFFFFu, 0xFFFFFFFFu } // KALIMBA<br>
> +<br>
> };<br>
><br>
> static const ArchDefinition g_elf_arch_def = {<br>
><br>
><br>
> 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.<br>
><br>
> Patch file attached.<br>
><br>
> thanks,<br>
> Matthew Gardiner<br>
><br>
><br>
> 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<br>
> More information can be found at <a href="http://www.csr.com" target="_blank">www.csr.com</a>. Keep up to date with CSR on our technical blog, <a href="http://www.csr.com/blog" target="_blank">www.csr.com/blog</a>, CSR people blog, <a href="http://www.csr.com/people" target="_blank">www.csr.com/people</a>, YouTube, <a href="http://www.youtube.com/user/CSRplc" target="_blank">www.youtube.com/user/CSRplc</a>, Facebook, <a href="http://www.facebook.com/pages/CSR/191038434253534" target="_blank">www.facebook.com/pages/CSR/191038434253534</a>, or follow us on Twitter at <a href="http://www.twitter.com/CSR_plc" target="_blank">www.twitter.com/CSR_plc</a>.<br>
> New for 2014, you can now access the wide range of products powered by aptX at <a href="http://www.aptx.com" target="_blank">www.aptx.com</a>.<br>
><br>
> _______________________________________________<br>
> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
><br>
><br>
><br>
><br>
> --<br>
> Todd Fiala | Software Engineer | <a href="mailto:tfiala@google.com">tfiala@google.com</a> | <a href="tel:650-943-3180" value="+16509433180">650-943-3180</a><br>
><br>
><br>
> _______________________________________________<br>
> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
><br>
><br>
><br>
><br>
> --<br>
> -Todd<br>
> _______________________________________________<br>
> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">-Todd</div>
</div>