Go ahead and revert to green until i get back<br><div class="gmail_quote"><div dir="ltr">On Thu, Nov 8, 2018 at 1:17 PM Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div dir="auto">+greg.</div><div dir="auto"><br></div><div dir="auto">Greg, is the test wrong here or the patch?  If it’s the test let’s just fix the test, otherwise we can revert the patch until we figure it out.</div><div dir="auto"><br></div><div dir="auto">It seems related to my change to use the layout type instead of the forward type.</div></div><div><br><div class="gmail_quote"><div dir="ltr">On Thu, Nov 8, 2018 at 1:03 PM Davide Italiano <<a href="mailto:dccitaliano@gmail.com" target="_blank">dccitaliano@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Nov 8, 2018 at 12:59 PM Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
><br>
> I’m ooo for at least 2 hours. Is it a test failure or a compilation failure?<br>
<br>
<br>
FAIL: test_with_run_command_gmodules (TestTypeCompletion.TypeCompletionTestCase)<br>
<br>
   Check that types only get completed when necessary.<br>
----------------------------------------------------------------------<br>
Traceback (most recent call last):<br>
  File "/Users/davide/work/llvm-project-20170507/lldb/packages/Python/lldbsuite/test/lldbtest.py",<br>
line 1744, in test_method<br>
    return attrvalue(self)<br>
  File "/Users/davide/work/llvm-project-20170507/lldb/packages/Python/lldbsuite/test/decorators.py",<br>
line 113, in wrapper<br>
    func(*args, **kwargs)<br>
  File "/Users/davide/work/llvm-project-20170507/lldb/packages/Python/lldbsuite/test/functionalities/type_completion/TestTypeCompletion.py",<br>
line 55, in test_with_run_command 'vector<T> complete but it should<br>
not be')<br>
AssertionError: True is not False : vector<T> complete but it should not be<br>
<br>
<br>
Do you want me to revert this in the meanwhile?<br>
<br>
--Davide<br>
</blockquote></div></div>
</blockquote></div>