Re: [Lldb-commits] [PATCH] D19548: Fix TestRegisterVariables.py on Windows

2016-04-26 Thread Adrian McCarthy via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL267616: Fix TestRegisterVariables.py on Windows (authored by amccarth). Changed prior to commit: http://reviews.llvm.org/D19548?vs=55050&id=55112#toc Repository: rL LLVM http://reviews.llvm.org/D195

Re: [Lldb-commits] [PATCH] D19548: Fix TestRegisterVariables.py on Windows

2016-04-26 Thread Todd Fiala via lldb-commits
tfiala accepted this revision. tfiala added a comment. This revision is now accepted and ready to land. Seems reasonable. We can revert and fix if this breaks something. http://reviews.llvm.org/D19548 ___ lldb-commits mailing list lldb-commits@list

[Lldb-commits] [PATCH] D19548: Fix TestRegisterVariables.py on Windows

2016-04-26 Thread Adrian McCarthy via lldb-commits
amccarth created this revision. amccarth added a reviewer: tfiala. amccarth added a subscriber: lldb-commits. Herald added a subscriber: aemerson. 32-bit Windows calling conventions, by default, don't pass arguments in registers, but this test expects at least one of them to be. By adding __attr