<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;">I have some cycles to look at this while I’m doing a build.<div><br></div><div>The original patch that brought this in is</div><div>–</div><div>Author: spyffe<br>Date: Wed Apr 13 21:01:31 2011<br>New Revision: 129500<br><br>URL: <a href="http://llvm.org/viewvc/llvm-project?rev=129500&view=rev">http://llvm.org/viewvc/llvm-project?rev=129500&view=rev</a><br>Log:<br>Updated LLVM to pick up fixes to the ARM instruction<br>tables.<br><br>Modified:<br> lldb/trunk/llvm.zip<br> lldb/trunk/scripts/build-llvm.pl<br> lldb/trunk/source/Expression/ClangExpressionParser.cpp<br> lldb/trunk/source/Expression/IRDynamicChecks.cpp<br> lldb/trunk/source/Expression/IRForTarget.cpp<br>–</div><div>At the time, I was working on the ARM/Thumb disassemblers. It may have been that at the time the target triples were in flux.</div><div>I don’t think there’s a specific use case that depends on this at this time, and if there is one, we can revisit it.</div><div>Go ahead and take this out.</div><div><br><div>
<div>Sean</div>
</div>
<br><div><blockquote type="cite"><div>On Jun 26, 2014, at 11:15 AM, Todd Fiala <<a href="mailto:tfiala@google.com">tfiala@google.com</a>> wrote:</div><br class="Apple-interchange-newline"><div>LGTM - here's how I tested:<br><br>Linux x86_64:<br>(lldb) file /path/to/exe<br>(lldb) b main.cpp:10<br>(lldb) run<br>... at breakpoint ...<br>(lldb) expr 5*3<br>// prints result as expected<br><br>Same result for MacOSX 10.9.3 with x86_64 MacOS executable.<br><br>Let's ping the Apple folks and see if there is some kind of simulator/device that make choke on that, though, before you check this in.<br><br><a href="http://reviews.llvm.org/D4282">http://reviews.llvm.org/D4282</a><br><br><br><br>_______________________________________________<br>lldb-commits mailing list<br>lldb-commits@cs.uiuc.edu<br>http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits<br></div></blockquote></div><br></div></body></html>