diff options
| author | Adrian McCarthy <amccarth@google.com> | 2016-02-29 21:13:29 +0000 |
|---|---|---|
| committer | Adrian McCarthy <amccarth@google.com> | 2016-02-29 21:13:29 +0000 |
| commit | 536183275d67c1de31066386070a2fe9b403e241 (patch) | |
| tree | 9c5d19fb559b9a2da11a3ddfc19bcad6199106cd /lldb/source/Plugins/Process/Windows/MiniDump/ProcessWinMiniDump.cpp | |
| parent | 936499ae4cdb4f4601e7c0561be4fdd8a05fb04e (diff) | |
| download | bcm5719-llvm-536183275d67c1de31066386070a2fe9b403e241.tar.gz bcm5719-llvm-536183275d67c1de31066386070a2fe9b403e241.zip | |
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
Diffstat (limited to 'lldb/source/Plugins/Process/Windows/MiniDump/ProcessWinMiniDump.cpp')
0 files changed, 0 insertions, 0 deletions
