Fix TestInlines.py on Windows
authorAdrian McCarthy <amccarth@google.com>
Mon, 29 Feb 2016 21:13:29 +0000 (21:13 +0000)
committerAdrian McCarthy <amccarth@google.com>
Mon, 29 Feb 2016 21:13:29 +0000 (21:13 +0000)
commit536183275d67c1de31066386070a2fe9b403e241
tree9c5d19fb559b9a2da11a3ddfc19bcad6199106cd
parent936499ae4cdb4f4601e7c0561be4fdd8a05fb04e
Fix TestInlines.py on Windows

The inlining semantics for C and C++ are different, which affects the test's expectation of the number of times the function should appear in the binary.  In the case of this test, C semantics means there should be three instances of inner_inline, while C++ semantics means there should be only two.

On Windows, clang uses C++ inline semantics even for C code, and there doesn't seem to be a combination of compiler flags to avoid this.

So, for consistency, I've recast the test to use C++ everywhere.  Since the test resided under lang/c, it seemed appropriate to move it to lang/cpp.

This does not address the other XFAIL for this test on Linux/gcc.  See https://llvm.org/bugs/show_bug.cgi?id=26710

Differential Revision: http://reviews.llvm.org/D17650

llvm-svn: 262255
lldb/packages/Python/lldbsuite/test/lang/cpp/inlines/Makefile [moved from lldb/packages/Python/lldbsuite/test/lang/c/inlines/Makefile with 67% similarity]
lldb/packages/Python/lldbsuite/test/lang/cpp/inlines/TestInlines.py [moved from lldb/packages/Python/lldbsuite/test/lang/c/inlines/TestInlines.py with 83% similarity]
lldb/packages/Python/lldbsuite/test/lang/cpp/inlines/inlines.cpp [moved from lldb/packages/Python/lldbsuite/test/lang/c/inlines/inlines.c with 100% similarity]
lldb/packages/Python/lldbsuite/test/lang/cpp/inlines/inlines.h [moved from lldb/packages/Python/lldbsuite/test/lang/c/inlines/inlines.h with 100% similarity]