<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 4, 2016 at 9:45 AM, William Dillon via swift-lldb-dev <span dir="ltr">&lt;<a href="mailto:swift-lldb-dev@swift.org" target="_blank">swift-lldb-dev@swift.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><span class=""><blockquote type="cite"><div>On Jan 4, 2016, at 9:17 AM, Todd Fiala &lt;<a href="mailto:tfiala@apple.com" target="_blank">tfiala@apple.com</a>&gt; wrote:</div><br><div><div style="word-wrap:break-word">I added some more comments to the pull request that I Think is related to this thread here:<div><a href="https://github.com/apple/swift-lldb/pull/3" target="_blank">https://github.com/apple/swift-lldb/pull/3</a></div><div><br></div><div>The archspec changes really need to happen in <a href="http://llvm.org/" target="_blank">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><br></div></div></div></blockquote><div><br></div></span><div>That sounds good.</div><div><br></div><div>To summarize how changes in these three projects link together:</div><div><br></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></div></blockquote><div><br></div><div>Sounds good.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div><br></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></div></blockquote><div><br></div><div>Okay.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div><br></div><div>Apologies for any incoherencies, I’ve got a cold at the moment. :)</div></div></div></blockquote><div><br></div><div>I hope you feel better!  Totally coherent fwiw :-)</div><div><br></div><div>-Todd</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div><br></div><div>- Will</div><span class=""><br><blockquote type="cite"><div><div style="word-wrap:break-word"><div><div><blockquote type="cite"><div>On Jan 4, 2016, at 9:00 AM, Dmitri Gribenko via swift-lldb-dev &lt;<a href="mailto:swift-lldb-dev@swift.org" target="_blank">swift-lldb-dev@swift.org</a>&gt; wrote:</div><br><div><div dir="ltr" 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"><div class="gmail_extra">On Mon, Jan 4, 2016 at 6:50 PM, Todd Fiala via swift-lldb-dev <span dir="ltr">&lt;<a href="mailto:swift-lldb-dev@swift.org" target="_blank">swift-lldb-dev@swift.org</a>&gt;</span> wrote:<br><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"><br><div><span><blockquote type="cite"><div>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">swift-lldb-dev@swift.org</a>&gt; wrote:</div><br><div><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">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"><br></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">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><br></div></span><div>I’d love to hear about the reliability piece if you have more details there!</div><span><br><blockquote type="cite"><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"> 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> </span><a href="http://lldb.llvm.org/" target="_blank">lldb.llvm.org</a> repo, rather than the swift one.</div></div></blockquote><div><br></div></span><div>I think I’ve seen the<span> </span><a href="http://llvm.org/" target="_blank">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><br><blockquote type="cite"><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"> 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><br></div></span><div>This is a great question that I am not the right resource to answer.  <b>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> </div><div>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><br></div><div>Dmitri</div></div><div><br></div>--<span> </span><br><div>main(i,j){for(i=2;;i++){for(j=2;j&lt;i;j++){if(!(i%j)){j=0;break;}}if<br>(j){printf(&quot;%d\n&quot;,i);}}} /*Dmitri Gribenko &lt;<a href="mailto:gribozavr@gmail.com" target="_blank">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;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;min-height:1px!important;width:1px!important;border-width:0px!important;margin:0px!important;padding:0px!important"><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"><span> </span></span><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">_______________________________________________</span><br 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"><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">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;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><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;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px" target="_blank">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;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><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;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px" target="_blank">https://lists.swift.org/mailman/listinfo/swift-lldb-dev</a></div></blockquote></div><br></div></div></div></blockquote></span></div><br>
<img src="https://u2002410.ct.sendgrid.net/wf/open?upn=rJF9aHCcP5KO6xYESYhGGW5DyMKCVwzfL2rfjdSdZ2tWKmqBn-2BfVL7UWOeBweGg8U4D-2BBIh4vujP-2BUjH8XcDTCn5XnuhM9-2FO2UMsgqvX8cCh0sj6SzFauliG3PG0fDyJXAtEIuuUZG-2Bysl5nGbXl3rzBX0T5oiGUv3UElBBfMAOT9q-2BVEy5JKLptgcOr9-2BtrguxNt5TzD8c2LUVGQXXAB3n9svJUUsEiFUnxLzuwKWw-3D" alt="" width="1" height="1" border="0" style="min-height:1px!important;width:1px!important;border-width:0!important;margin-top:0!important;margin-bottom:0!important;margin-right:0!important;margin-left:0!important;padding-top:0!important;padding-bottom:0!important;padding-right:0!important;padding-left:0!important">
</div>
<br>_______________________________________________<br>
swift-lldb-dev mailing list<br>
<a href="mailto:swift-lldb-dev@swift.org">swift-lldb-dev@swift.org</a><br>
<a href="https://lists.swift.org/mailman/listinfo/swift-lldb-dev" rel="noreferrer" target="_blank">https://lists.swift.org/mailman/listinfo/swift-lldb-dev</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">-Todd</div></div>
</div></div>