<div dir="ltr">I made a quick investigation today. The problem is indeed specific to gcc-4.9. The main differences I could see is that gcc-4.9 generates a different prologue for the function, and uses pushl for argument passing. This confuses the emulator/augmenter and the unwind information at some points ends up being incorrect. Since the problem starts to occur after the first pushl instruction, i am guessing the problem is with those. I'll try to investigate more next week.<div><br></div><div> For reference, the function with 4.9 looks like this:<div><br></div><div>
<p class="gmail-p1"><span class="gmail-s1">a.out`main:</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80484eb <+0>: leal 0x4(%esp), %ecx</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80484ef <+4>: andl $-0x10, %esp</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80484f2 <+7>: pushl -0x4(%ecx)</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80484f5 <+10>: pushl %ebp</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80484f6 <+11>: movl %esp, %ebp</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80484f8 <+13>: pushl %edi</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80484f9 <+14>: pushl %esi</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80484fa <+15>: pushl %ebx</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80484fb <+16>: pushl %ecx</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80484fc <+17>: subl $0x28, %esp</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80484ff <+20>: subl $0xc, %esp</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048502 <+23>: pushl $0x18</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048504 <+25>: calll 0x80483d0 ; symbol stub for: operator new[](unsigned int)</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048509 <+30>: addl $0x10, %esp</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804850c <+33>: movl %eax, %edi</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804850e <+35>: movl %edi, %eax</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048510 <+37>: movl $0x1, %esi</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048515 <+42>: movl %eax, %ebx</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048517 <+44>: jmp 0x804852b ; <+64> at main.cpp:26</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048519 <+46>: subl $0xc, %esp</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804851c <+49>: pushl %ebx</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804851d <+50>: calll 0x80485cc ; foo::foo at main.cpp:16</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048522 <+55>: addl $0x10, %esp</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048525 <+58>: addl $0xc, %ebx</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048528 <+61>: subl $0x1, %esi</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804852b <+64>: cmpl $-0x1, %esi</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804852e <+67>: jne 0x8048519 ; <+46> at main.cpp:26</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048530 <+69>: movl %edi, -0x1c(%ebp)</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048533 <+72>: movl -0x1c(%ebp), %eax</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048536 <+75>: movl $0x1, (%eax)</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804853c <+81>: movl -0x1c(%ebp), %eax</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804853f <+84>: movl $0x9, 0x4(%eax)</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048546 <+91>: movl -0x1c(%ebp), %eax</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048549 <+94>: addl $0xc, %eax</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804854c <+97>: movl $0x8, (%eax)</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048552 <+103>: movl -0x1c(%ebp), %eax</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048555 <+106>: addl $0xc, %eax</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048558 <+109>: movl $0x5, 0x4(%eax)</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804855f <+116>: movl -0x1c(%ebp), %eax</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048562 <+119>: addl $0xc, %eax</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048565 <+122>: movl $0x7, 0x4(%eax)</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804856c <+129>: movl -0x1c(%ebp), %eax</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804856f <+132>: leal 0xc(%eax), %edx</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048572 <+135>: movl 0x80486a4, %eax</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048577 <+140>: movl %eax, 0x8(%edx)</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804857a <+143>: leal -0x34(%ebp), %ebx</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804857d <+146>: movl 0x80486a8, %eax</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048582 <+151>: pushl %eax</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048583 <+152>: pushl $0x2</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048585 <+154>: pushl $0x1</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048587 <+156>: pushl %ebx</span></p>
<p class="gmail-p1"><span class="gmail-s1">-> 0x8048588 <+157>: calll 0x80485f0 ; foo::foo at main.cpp:17</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804858d <+162>: addl $0x10, %esp</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048590 <+165>: leal 0xc(%ebx), %edx</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048593 <+168>: movl 0x80486ac, %eax</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048598 <+173>: pushl %eax</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x8048599 <+174>: pushl $0x5</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804859b <+176>: pushl $0x4</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804859d <+178>: pushl %edx</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x804859e <+179>: calll 0x80485f0 ; foo::foo at main.cpp:17</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80485a3 <+184>: addl $0x10, %esp</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80485a6 <+187>: cmpl $0x0, -0x1c(%ebp)</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80485aa <+191>: je 0x80485ba ; <+207> at main.cpp:42</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80485ac <+193>: subl $0xc, %esp</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80485af <+196>: pushl -0x1c(%ebp)</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80485b2 <+199>: calll 0x80483e0 ; symbol stub for: operator delete[](void*)</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80485b7 <+204>: addl $0x10, %esp</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80485ba <+207>: movl $0x0, %eax</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80485bf <+212>: leal -0x10(%ebp), %esp</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80485c2 <+215>: popl %ecx</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80485c3 <+216>: popl %ebx</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80485c4 <+217>: popl %esi</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80485c5 <+218>: popl %edi</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80485c6 <+219>: popl %ebp</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80485c7 <+220>: leal -0x4(%ecx), %esp</span></p>
<p class="gmail-p1"><span class="gmail-s1"> 0x80485ca <+223>: retl </span></p><p class="gmail-p1"><span class="gmail-s1"><br></span></p><p class="gmail-p1"><span class="gmail-s1"><br></span></p><p class="gmail-p1"><span class="gmail-s1">whereas the same function compiled with gcc-4.8 is this:</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x804850d <+0>: pushl %ebp</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x804850e <+1>: movl %esp, %ebp</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048510 <+3>: pushl %edi</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048511 <+4>: pushl %esi</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048512 <+5>: pushl %ebx</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048513 <+6>: andl $-0x10, %esp</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048516 <+9>: subl $0x30, %esp</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048519 <+12>: movl $0x18, (%esp)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048520 <+19>: calll 0x80483f0 ; symbol stub for: operator new[](unsigned int)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048525 <+24>: movl %eax, %edi</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048527 <+26>: movl %edi, %eax</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048529 <+28>: movl $0x1, %esi</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x804852e <+33>: movl %eax, %ebx</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048530 <+35>: jmp 0x8048540 ; <+51> at main.cpp:26</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048532 <+37>: movl %ebx, (%esp)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048535 <+40>: calll 0x8048600 ; foo::foo at main.cpp:16</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x804853a <+45>: addl $0xc, %ebx</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x804853d <+48>: subl $0x1, %esi</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048540 <+51>: cmpl $-0x1, %esi</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048543 <+54>: jne 0x8048532 ; <+37> at main.cpp:26</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048545 <+56>: movl %edi, 0x14(%esp)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048549 <+60>: movl 0x14(%esp), %eax</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x804854d <+64>: movl $0x1, (%eax)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048553 <+70>: movl 0x14(%esp), %eax</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048557 <+74>: movl $0x9, 0x4(%eax)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x804855e <+81>: movl 0x14(%esp), %eax</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048562 <+85>: addl $0xc, %eax</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048565 <+88>: movl $0x8, (%eax)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x804856b <+94>: movl 0x14(%esp), %eax</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x804856f <+98>: addl $0xc, %eax</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048572 <+101>: movl $0x5, 0x4(%eax)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048579 <+108>: movl 0x14(%esp), %eax</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x804857d <+112>: addl $0xc, %eax</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048580 <+115>: movl $0x7, 0x4(%eax)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048587 <+122>: movl 0x14(%esp), %eax</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x804858b <+126>: leal 0xc(%eax), %edx</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x804858e <+129>: movl 0x80486e4, %eax</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048593 <+134>: movl %eax, 0x8(%edx)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x8048596 <+137>: leal 0x18(%esp), %ebx</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x804859a <+141>: movl 0x80486e8, %eax</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x804859f <+146>: movl %eax, 0xc(%esp)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485a3 <+150>: movl $0x2, 0x8(%esp)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485ab <+158>: movl $0x1, 0x4(%esp)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485b3 <+166>: movl %ebx, (%esp)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485b6 <+169>: calll 0x8048624 ; foo::foo at main.cpp:17</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485bb <+174>: leal 0xc(%ebx), %edx</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485be <+177>: movl 0x80486ec, %eax</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485c3 <+182>: movl %eax, 0xc(%esp)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485c7 <+186>: movl $0x5, 0x8(%esp)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485cf <+194>: movl $0x4, 0x4(%esp)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485d7 <+202>: movl %edx, (%esp)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485da <+205>: calll 0x8048624 ; foo::foo at main.cpp:17</span></p><p class="gmail-p1"><span class="gmail-s1">-> 0x80485df <+210>: cmpl $0x0, 0x14(%esp)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485e4 <+215>: je 0x80485f2 ; <+229> at main.cpp:42</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485e6 <+217>: movl 0x14(%esp), %eax</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485ea <+221>: movl %eax, (%esp)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485ed <+224>: calll 0x8048400 ; symbol stub for: operator delete[](void*)</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485f2 <+229>: movl $0x0, %eax</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485f7 <+234>: leal -0xc(%ebp), %esp</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485fa <+237>: popl %ebx</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485fb <+238>: popl %esi</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485fc <+239>: popl %edi</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485fd <+240>: popl %ebp</span></p><p class="gmail-p1"><span class="gmail-s1">
</span></p><p class="gmail-p1"><span class="gmail-s1"> 0x80485fe <+241>: retl </span></p></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 29 September 2016 at 14:22, Jason Molenda <span dir="ltr"><<a href="mailto:jmolenda@apple.com" target="_blank">jmolenda@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Interesting, I can't get the failure to repo with i386 + gcc-4.8 on an ubuntu 14.04.5 system. Downloading & building gcc-4.9.4 right now.<br>
<br>
<br>
<br>
$ ./dotest.py -C gcc-4.8 -A i386 --executable ~/build/bin/lldb testcases/python_api/sbdata<br>
LLDB library dir: /home/jmolenda/build/bin<br>
LLDB import library dir: /home/jmolenda/build/bin<br>
lldb version 4.0.0 (<a href="http://llvm.org/svn/llvm-project/lldb/trunk" rel="noreferrer" target="_blank">http://llvm.org/svn/llvm-<wbr>project/lldb/trunk</a> revision 282753 clang revision 282567 llvm revision 282567)<br>
The 'lldb-mi' executable cannot be located. The lldb-mi tests can not be run as a result.<br>
<br>
Session logs for test failures/errors/unexpected successes will go into directory '2016-09-29-15_09_05'<br>
Command invoked: ./dotest.py -C gcc-4.8 -A i386 --executable /home/jmolenda/build/bin/lldb testcases/python_api/sbdata<br>
compilers=['gcc-4.8']<br>
<br>
Configuration: arch=i386 compiler=gcc-4.8<br>
------------------------------<wbr>------------------------------<wbr>----------<br>
Collected 2 tests<br>
<br>
------------------------------<wbr>------------------------------<wbr>----------<br>
Ran 2 tests in 1.540s<br>
<br>
RESULT: PASSED (2 passes, 0 failures, 0 errors, 0 skipped, 0 expected failures, 0 unexpected successes)<br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
J<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> On Sep 29, 2016, at 12:29 PM, Jason Molenda <<a href="mailto:jmolenda@apple.com">jmolenda@apple.com</a>> wrote:<br>
><br>
> Yep, will do. I saw that build bot result last night and ran the testsuite on my local ubuntu box and didn't repo the failure so I thought maybe it was an already-failing test case that the bot was just telling me about. But I think I was running the test x86_64 - I'll figure out how to run it i386 and look into it.<br>
><br>
>> On Sep 29, 2016, at 5:56 AM, Pavel Labath <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br>
>><br>
>> Note that the test fails when using gcc as a compiler (specifically gcc-4.9 in this case, but hopefully the exact version does not matter here).<br>
>><br>
>> Jason, will you be able to check this out today?<br>
>><br>
>> On 29 September 2016 at 05:45, Dimitar Vlahovski via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org">lldb-commits@lists.llvm.org</a>> wrote:<br>
>> This is the first build that failed right after your CL: <a href="http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake/builds/20083" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/<wbr>builders/lldb-x86_64-ubuntu-<wbr>14.04-cmake/builds/20083</a><br>
>><br>
>> On Thu, Sep 29, 2016 at 1:35 PM, Dimitar Vlahovski <<a href="mailto:dvlahovski@google.com">dvlahovski@google.com</a>> wrote:<br>
>> Hi,<br>
>><br>
>> Is the work that you are currently doing the reason why the lldb build on i386 is failing?<br>
>> <a href="http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/<wbr>builders/lldb-x86_64-ubuntu-<wbr>14.04-cmake</a><br>
>> <a href="http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake/builds/20099" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/<wbr>builders/lldb-x86_64-ubuntu-<wbr>14.04-cmake/builds/20099</a><br>
>><br>
>> Dimitar<br>
>><br>
>><br>
>> On Thu, Sep 29, 2016 at 5:30 AM, Jason Molenda via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org">lldb-commits@lists.llvm.org</a>> wrote:<br>
>> Good suggestions, thanks. I'll fix those when I commit the 32-bit version of the same test.<br>
>><br>
>> J<br>
>><br>
>>> On Sep 28, 2016, at 9:28 PM, Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br>
>>><br>
>>><br>
>>><br>
>>> On Wed, Sep 28, 2016 at 9:10 PM Jason Molenda via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org">lldb-commits@lists.llvm.org</a>> wrote:<br>
>>><br>
>>> + EXPECT_TRUE(regloc.GetOffset() == -8);<br>
>>> This should be<br>
>>><br>
>>> EXPECT_EQ(-8, regloc.GetOffset());<br>
>>><br>
>>> That way if it fails, you'll get a handy error message that says:<br>
>>><br>
>>> Expected: -8<br>
>>> Actual: -7<br>
>>><br>
>>> If you use EXPECT_TRUE, it's not going to tell you the actual value. The same goes for many other places in the file. Note that you're supposed to put the expected value *first*. The test is the same either way obviously, but it affects the printing of the above message.<br>
>>><br>
>>> +<br>
>>> + // these could be set to IsSame and be valid -- meaning that the<br>
>>> + // register value is the same as the caller's -- but I'd rather<br>
>>> + // they not be mentioned at all.<br>
>>> + EXPECT_TRUE(row_sp-><wbr>GetRegisterInfo(k_rbp, regloc) == false);<br>
>>> + EXPECT_TRUE(row_sp-><wbr>GetRegisterInfo(k_r15, regloc) == false);<br>
>>> + EXPECT_TRUE(row_sp-><wbr>GetRegisterInfo(k_r14, regloc) == false);<br>
>>> + EXPECT_TRUE(row_sp-><wbr>GetRegisterInfo(k_r13, regloc) == false);<br>
>>> + EXPECT_TRUE(row_sp-><wbr>GetRegisterInfo(k_r12, regloc) == false);<br>
>>> + EXPECT_TRUE(row_sp-><wbr>GetRegisterInfo(k_rbx, regloc) == false);<br>
>>> If you're using EXPECT_TRUE and EXPECT_FALSE, I think it's more intuitive to not use the comparison operator. The above is just<br>
>>><br>
>>> EXPECT_FALSE(row_sp-><wbr>GetRegisterInfo(k_rbx, regloc));<br>
>><br>
>> ______________________________<wbr>_________________<br>
>> lldb-commits mailing list<br>
>> <a href="mailto:lldb-commits@lists.llvm.org">lldb-commits@lists.llvm.org</a><br>
>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/lldb-commits</a><br>
>><br>
>><br>
>><br>
>> ______________________________<wbr>_________________<br>
>> lldb-commits mailing list<br>
>> <a href="mailto:lldb-commits@lists.llvm.org">lldb-commits@lists.llvm.org</a><br>
>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/lldb-commits</a><br>
>><br>
>><br>
><br>
<br>
</div></div></blockquote></div><br></div>