<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Jan 4, 2016, at 9:17 AM, Todd Fiala &lt;<a href="mailto:tfiala@apple.com" class="">tfiala@apple.com</a>&gt; wrote:</div><br class="Apple-interchange-newline"><div class=""><meta http-equiv="Content-Type" content="text/html charset=utf-8" class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">I added some more comments to the pull request that I Think is related to this thread here:<div class=""><a href="https://github.com/apple/swift-lldb/pull/3" class="">https://github.com/apple/swift-lldb/pull/3</a></div><div class=""><br class=""></div><div class="">The archspec changes really need to happen in <a href="http://llvm.org/" class="">llvm.org</a>&nbsp;lldb svn trunk and need to be put up for patch review there (along with any discussion particular to that). &nbsp;The rest of the change (for 32-bit support specific to Swift LLDB) looked fine. If we get that separated out again, I’ll be happy to get that in.</div><div class=""><br class=""></div></div></div></blockquote><div><br class=""></div><div>That sounds good.</div><div><br class=""></div><div>To summarize how changes in these three projects link together:</div><div><br class=""></div><div>there is some code in swift that tries to canonicalize armv6l and armv7l into just ‘armv6’ and ‘armv7’ respectively. &nbsp;This was suggested by Dmitri and Tim. &nbsp;However, armv6l isn’t represented in llvm, and I’m working on bringing my patch there up to standards. &nbsp;However, this canonicalization only applies to compilation, not the REPL. &nbsp;Joe Bell figured out that LLDB was being passed armv6l and armv7l and it didn’t know what to do with them. &nbsp;He fixed that by adding them to lldb. &nbsp;The correct approach seems to be performing the same transformation for the REPL as for compilation, and therefore the other proposed changes to LLDB are inappropriate. &nbsp;I’ll back those out as Todd suggested.</div><div><br class=""></div><div>I want to do these operations in such a way that Joe always has a set of repos that will build functioning binaries for ARM because he’s been great at releasing updates as other parts of swift for linux mature. &nbsp;To that end, I’ll figure out the changes necessary to swift’s invocation of LLDB and submit a PR for that. &nbsp;Then, I’ll back-out the changes to LLDB. &nbsp;The changes to LLVM can happen any time.</div><div><br class=""></div><div>Apologies for any incoherencies, I’ve got a cold at the moment. :)</div><div><br class=""></div><div>- Will</div><br class=""><blockquote type="cite" class=""><div class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class=""><div class=""><blockquote type="cite" class=""><div class="">On Jan 4, 2016, at 9:00 AM, Dmitri Gribenko via swift-lldb-dev &lt;<a href="mailto:swift-lldb-dev@swift.org" class="">swift-lldb-dev@swift.org</a>&gt; wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><div class="gmail_extra">On Mon, Jan 4, 2016 at 6:50 PM, Todd Fiala via swift-lldb-dev&nbsp;<span dir="ltr" class="">&lt;<a href="mailto:swift-lldb-dev@swift.org" target="_blank" class="">swift-lldb-dev@swift.org</a>&gt;</span>&nbsp;wrote:<br class=""><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex;"><div style="word-wrap: break-word;" class=""><br class=""><div class=""><span class=""><blockquote type="cite" class=""><div class="">On Dec 28, 2015, at 10:39 AM, William Dillon via swift-lldb-dev &lt;<a href="mailto:swift-lldb-dev@swift.org" target="_blank" class="">swift-lldb-dev@swift.org</a>&gt; wrote:</div><br class=""><div class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; float: none; display: inline !important;" class="">Hi Todd,</span><div style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;" class=""><br class=""></div><div style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;" class="">Yes, I think that LLDB is more or less working with Swift on ARM.&nbsp; We can start the REPL and do some tasks with it, though it isn’t all that reliable yet.</div></div></blockquote><div class=""><br class=""></div></span><div class="">I’d love to hear about the reliability piece if you have more details there!</div><span class=""><br class=""><blockquote type="cite" class=""><div class=""><div style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;" class="">&nbsp;There are two files in the swift-lldb PR that I merged in from Joe Bell that fixed the REPL.&nbsp; I think, however, than they would need to go to the<span class="">&nbsp;</span><a href="http://lldb.llvm.org/" target="_blank" class="">lldb.llvm.org</a>&nbsp;repo, rather than the swift one.</div></div></blockquote><div class=""><br class=""></div></span><div class="">I think I’ve seen the<span class="Apple-converted-space">&nbsp;</span><a href="http://llvm.org/" target="_blank" class="">llvm.org</a>&nbsp;side of that patch.&nbsp; Somebody else and I had a question about some static casts in there, but we’ll catch up on that side if it is the same patch I’m thinking about.</div><span class=""><br class=""><blockquote type="cite" class=""><div class=""><div style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;" class="">&nbsp;There is a question about that, though.&nbsp; Joe added armv7l into the ArchSpec table, but I don’t think that&nbsp;armv7l&nbsp;is a real subtype.&nbsp; I’ve been very confused by the ARM nomenclature on linux (I don’t think I’m alone here), and I think that armv7l means armv7 little endian.&nbsp; So, should these get converted to armv7 somewhere else and revert the changes to the ArchSpec table?</div></div></blockquote><div class=""><br class=""></div></span><div class="">This is a great question that I am not the right resource to answer. &nbsp;<b class="">Greg or Jason,</b>&nbsp;any thoughts here on question of arm subtypes? &nbsp;(My first take is that I’d expect each subtype to represent a different set of features and instructions available, and not be used solely for an endianness designation, but that’s not an area I do much work in).</div></div></div></blockquote><div class="">&nbsp;</div><div class="">We have discussed this before with Tim Northover, and he suggested that we canonicalize armv7l to armv7 when parsing the triple from the command line.</div><div class=""><br class=""></div><div class="">Dmitri</div></div><div class=""><br class=""></div>--<span class="Apple-converted-space">&nbsp;</span><br class=""><div class="gmail_signature">main(i,j){for(i=2;;i++){for(j=2;j&lt;i;j++){if(!(i%j)){j=0;break;}}if<br class="">(j){printf("%d\n",i);}}} /*Dmitri Gribenko &lt;<a href="mailto:gribozavr@gmail.com" target="_blank" class="">gribozavr@gmail.com</a>&gt;*/</div></div></div><img src="https://u2002410.ct.sendgrid.net/wf/open?upn=ehWBPZdpvgOgh-2FSXym9sUWqjOEmupm2f-2FeV2NzZZAiSDd6AqyVR1oAGXp-2FPOGCuZQ-2BAnsEUHkRI-2Ffb76t4GWHXjH74wYI9nRjEUuZ4BJcnUjFFs1w4u4JnXpl1LndU7baufFp-2F93JFn5-2FB8lw8SeISOMv6t69GhE3-2FGli8FA7lP-2BFXHb3affB-2Bn-2B4Dm-2Fpj5psqB5nY-2BreoWC37OxE5RuOKgUnDGJWQAhOaNaApEHBXE-3D" alt="" width="1" height="1" border="0" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; height: 1px !important; width: 1px !important; border-width: 0px !important; margin: 0px !important; padding: 0px !important;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class=""><span class="Apple-converted-space">&nbsp;</span></span><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">_______________________________________________</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">swift-lldb-dev mailing list</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><a href="mailto:swift-lldb-dev@swift.org" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">swift-lldb-dev@swift.org</a><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><a href="https://lists.swift.org/mailman/listinfo/swift-lldb-dev" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">https://lists.swift.org/mailman/listinfo/swift-lldb-dev</a></div></blockquote></div><br class=""></div></div></div></blockquote></div><br class=""></body></html>