<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 <<a href="mailto:tfiala@apple.com" class="">tfiala@apple.com</a>> 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> lldb svn trunk and need to be put up for patch review there (along with any discussion particular to that). 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. This was suggested by Dmitri and Tim. However, armv6l isn’t represented in llvm, and I’m working on bringing my patch there up to standards. However, this canonicalization only applies to compilation, not the REPL. Joe Bell figured out that LLDB was being passed armv6l and armv7l and it didn’t know what to do with them. He fixed that by adding them to lldb. 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. 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. To that end, I’ll figure out the changes necessary to swift’s invocation of LLDB and submit a PR for that. Then, I’ll back-out the changes to LLDB. 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 <<a href="mailto:swift-lldb-dev@swift.org" class="">swift-lldb-dev@swift.org</a>> 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 <span dir="ltr" class=""><<a href="mailto:swift-lldb-dev@swift.org" target="_blank" class="">swift-lldb-dev@swift.org</a>></span> 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 <<a href="mailto:swift-lldb-dev@swift.org" target="_blank" class="">swift-lldb-dev@swift.org</a>> 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. 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=""> There are two files in the swift-lldb PR that I merged in from Joe Bell that fixed the REPL. I think, however, than they would need to go to the<span class=""> </span><a href="http://lldb.llvm.org/" target="_blank" class="">lldb.llvm.org</a> 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"> </span><a href="http://llvm.org/" target="_blank" class="">llvm.org</a> side of that patch. 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=""> There is a question about that, though. Joe added armv7l into the ArchSpec table, but I don’t think that armv7l is a real subtype. 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. 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. <b class="">Greg or Jason,</b> any thoughts here on question of arm subtypes? (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=""> </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"> </span><br class=""><div class="gmail_signature">main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if<br class="">(j){printf("%d\n",i);}}} /*Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com" target="_blank" class="">gribozavr@gmail.com</a>>*/</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"> </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>