summaryrefslogtreecommitdiffstats
path: root/lldb
Commit message (Collapse)AuthorAgeFilesLines
...
* [CMake] Don't include LLDB_TEST_COMPILER in cached variableChris Bieneman2016-10-181-2/+8
| | | | | | | | | | | | | | | Summary: CMake has no builtin mechanism for cache invalidation. As a general convention you want to not expand user-specified variables in other cached variables because they will not get updated when the user changes their specified value. This patch moves the "-C" option for dotest.py into the LLDB_TEST_COMMON_ARGS and out of the CMake cache. In order to prevent issues with out-of-date cache files on builders I've added code to scrub "-C ${LLDB_TEST_COMPILER}" out of the CMake caches, by Force writing the variable. This code can be removed in a few days once the change has trickled through CI systems. Reviewers: tfiala, labath, zturner Subscribers: lldb-commits, mgorny Differential Revision: https://reviews.llvm.org/D25751 llvm-svn: 284551
* Use clang --driver-mode instead of guessing c++ compiler pathChris Bieneman2016-10-181-18/+21
| | | | | | | | | | | | | | | Summary: When building the LLDB test programs, if your CC is clang it actually isn't safe to make CXX a string replace of "clang -> clang++". This falls down on unix configurations if your compiler is clang-${version}. A safer approach is to use the "--driver-mode=g++" option to tell clang to act like clang++. Reviewers: tfiala, zturner, labath Subscribers: lldb-commits Differential Revision: https://reviews.llvm.org/D25753 llvm-svn: 284550
* xfail TestMiSyntax.py's test_lldbmi_output_grammar on macOSTodd Fiala2016-10-181-0/+1
| | | | | | | | | Needs to be investigated. This is failing locally and on the Xcode CI. rdar://28805064 llvm-svn: 284484
* Fixup r284466 - try to unbreak NetBSDPavel Labath2016-10-181-3/+4
| | | | | | | NetBSD does not have getopt as well - we need to apply the workaround there too. FreeBSD seems to be fine though. llvm-svn: 284469
* [cmake] Make dependencies of lldb libraries private, take 2Pavel Labath2016-10-185-15/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: The dependencies of our libraries (only liblldb, really) we marked as public, which caused all their dependencies to be repeated when linking any executables to them. This is a problem because then all the .a files could end up being linked twice, once to liblldb and once again to to the executable linking against liblldb (lldb, lldb-mi). As it turns out, our build actually depends on this behavior: - on windows, lldb does not have getopt, so it pulls it from inside liblldb, even though getopt is not a part of the exported interface of liblldb (maybe some of the bsd variants have this problem as well) - lldb-mi uses llvm, which again is not exported by liblldb This change does not actually fix these problems (that is going to be a hard one), but it does make them explicit by moving this magic from add_lldb_library to the places the executable targets are defined. That way, I can link the additional .a files only on targets that really need it, and the other targets can build cleanly and make sure we don't regress further. It also fixes the LLVM_LINK_LLVM_DYLIB build on linux. Reviewers: zturner, beanz Subscribers: ki.stfu, lldb-commits, mgorny Differential Revision: https://reviews.llvm.org/D25680 llvm-svn: 284466
* Remove a debug print statement.Jim Ingham2016-10-181-1/+0
| | | | llvm-svn: 284448
* More testsuite xfail markings cleanup.Jim Ingham2016-10-182-3/+2
| | | | llvm-svn: 284446
* Fix a crash in expressions with fixits in the dummy target.Jim Ingham2016-10-172-2/+12
| | | | | | | | | In the expression command, if the target is NULL, you have to use the dummy target. <rdar://problem/28811687> llvm-svn: 284439
* [lldb] Read modules from memory when a local copy is not availableWalter Erquinigo2016-10-172-69/+113
| | | | | | | | | | | | | | | | | | | Summary: When the local lldb doesn't have access to a copy of the modules in the target, e.g. winphone, with this change now we read these modules from memory. There are mainly 2 changes: 1. create pecoff object files from memory 2. read from memory when the local file is not available Reviewers: sas, fjricci, zturner Subscribers: #lldb Tags: #lldb Differential Revision: https://reviews.llvm.org/D24284 llvm-svn: 284422
* unittests: Specify types in a bunch of unittest EXPECT'sJustin Bogner2016-10-176-178/+178
| | | | | | | The EXPECT and ASSERT macros in gtest don't do the usual arithmetic conversions. Specify types in several of them to fix -Werror. llvm-svn: 284405
* Interpreter: Don't return StringRef from functions whose return value is ↵Justin Bogner2016-10-172-31/+16
| | | | | | | | | | never used StringRef is passed through all of these APIs but never actually used. Just remove it from the API for now and if people want to use it they can add it back. llvm-svn: 284362
* This test is no longer failing for gmodules.Jim Ingham2016-10-151-4/+0
| | | | llvm-svn: 284296
* [CMake] Populate LLDB.framework's headers directoryChris Bieneman2016-10-141-1/+17
| | | | | | | | | | | | | | | | | Summary: This patch adds support for installing public headers in LLDB.framework, and symlinking the headers into the build directory. While writing the patch I discovered a bug in CMake that prevents applying POST_BUILD steps to framework targets (https://gitlab.kitware.com/cmake/cmake/issues/16363). I've implemented the support using POST_BUILD steps wrapped under a CMake version check with a TODO so that we can track the fix. Reviewers: tfiala, zturner, spyffe Subscribers: lldb-commits, mgorny Differential Revision: https://reviews.llvm.org/D25570 llvm-svn: 284250
* [LLDB-MI] Minor cleanup of CMICmnLLDBUtilSBValue classVadim Macagon2016-10-142-16/+17
| | | | | | | | | | | | | Summary: Placeholder c-strings don't need to be instance variables. Reviewers: ki.stfu, abidh Subscribers: lldb-commits Differential Revision: https://reviews.llvm.org/D25592 llvm-svn: 284231
* This test passes on i386 now.Jim Ingham2016-10-141-4/+0
| | | | llvm-svn: 284183
* This test is passing on i386 now.Jim Ingham2016-10-141-2/+0
| | | | llvm-svn: 284182
* [lldb] Improve identification of Linux core dumps. Fix for bug #30485.Richard Chamberlain2016-10-131-2/+1
| | | | | | | | | | | | | | | Summary: ObjectFileELF::RefineModuleDetailsFromNote() identifies Linux core dumps by searching for library paths starting with /lib/x86_64-linux-gnu or /lib/i386-linux-gnu. This change widens the test to allow for linux installations which have addition directories in the path. Reviewers: ted, hhellyer, clayborg Subscribers: lldb-commits Differential Revision: https://reviews.llvm.org/D25179 llvm-svn: 284114
* Fix Python binding generation build step on WindowsVadim Macagon2016-10-131-4/+4
| | | | | | | | | | | | | | | | | | | | Summary: If Python is installed to a location that contains spaces (e.g. "C:\Program Files\Python3") then the build fails while attempting to run the modify-python-lldb.py script because the path to the Python executable is not double-quoted before being passed to the shell. The fix consists of letting Python handle the formatting of the command line, since subprocess.Popen() is perfectly capable of handling paths containing spaces if it's given the command and arguments as a list instead of a single pre-formatted string. Reviewers: zturner, clayborg Subscribers: lldb-commits Differential Revision: https://reviews.llvm.org/D25396 llvm-svn: 284100
* [CMake] Cleanup check-lldb targetsChris Bieneman2016-10-121-2/+17
| | | | | | | | | | | | | | | | | Summary: This patch adds the following fixes to the check-lldb targets: * Adds missing dependencies on lldb tools so they get built before tests execute * Adds Ninja USES_TERMINAL to the target so that the output streams to stdout as it executes * Uses a generator expression to find the lldb executable, this is more robust than constructing the path manually Reviewers: tfiala, zturner Subscribers: mgorny, lldb-commits Differential Revision: https://reviews.llvm.org/D25490 llvm-svn: 284046
* Use LLDB_SRC for relative pathsChris Bieneman2016-10-121-2/+10
| | | | | | | | | | | | | | | Summary: Going from LLDB_SRC instead of the file path is safer when looking for compiler-rt. Also need to add support for looking inside the LLVM runtimes subdirectory. Eventually we should just get CMake to provide these paths during configuration. Reviewers: tfiala, zturner Subscribers: lldb-commits Differential Revision: https://reviews.llvm.org/D25489 llvm-svn: 284045
* Fix test suite lookup path for LLDB.hChris Bieneman2016-10-121-7/+9
| | | | | | | | | | | | | | | | | Summary: When running on Darwin, the test suite assumes a specific directory structure for the build directory. This works for the Xcode project builds, but fails for CMake builds regardless of whether or not you are generating the LLDB framework. This patch allows the Darwin code path to fall back to the more generic code path used by other platforms in the event that LLDB.h isn't where the test suite expects it. This allows API tests to run on Darwin when building with CMake with the framework build enabled or disabled. Reviewers: tfiala, zturner Subscribers: labath, lldb-commits Differential Revision: https://reviews.llvm.org/D25488 llvm-svn: 284043
* Fix building tests without system headers on DarwinChris Bieneman2016-10-121-1/+1
| | | | | | | | | | | | Summary: Default installations of OS X do not have system headers installed at /usr/include. This patch allows the LLDB test executables to properly compile when built on a system without headers at /usr/include by specifying a default value for the apple-sdk flag as "macosx". Reviewers: tfiala, zturner Subscribers: lldb-commits Differential Revision: https://reviews.llvm.org/D25487 llvm-svn: 284042
* Fix lookup path for lldb-miChris Bieneman2016-10-121-6/+7
| | | | | | | | | | | | | | | | | Summary: The test suite calls realpath on the lldb executable then append "-mi" to it to find the path of the lldb-mi executable. This does not work when using CMake builds on *nix platforms. On *nix platforms when a version number is set on executables CMake generates the binary as ${name}-${version} with a symlink named ${name} pointing to it. This results in the lldb executable being named lldb-4.0.0, and since lldb-4.0.0-mi doesn't ever match the lldb-mi executable these tests are always disabled. This patch looks for lldb-mi in the same directory as lldb. Reviewers: zturner, tfiala Subscribers: ki.stfu, enlight, lldb-commits Differential Revision: https://reviews.llvm.org/D25486 llvm-svn: 284041
* [LLDB][MIPS] fix Floating point register read/write for big endianNitesh Jain2016-10-125-260/+331
| | | | | | | | | | Reviewers: clayborg, labath, jaydeep Subscribers: bhushan, slthakur, lldb-commits Differential Revision: https://reviews.llvm.org/D24603 llvm-svn: 284003
* [LLDB][MIPS] Fix qProcessInfo to return correct pointer size based on ELF ABINitesh Jain2016-10-124-8/+58
| | | | | | | | | | Reviewers: clayborg, labath Subscribers: jaydeep, bhushan, slthakur, lldb-commits Differential Revision: https://reviews.llvm.org/D25021 llvm-svn: 284001
* Add an arm64 unit test where the function saves x20 in theJason Molenda2016-10-121-1/+101
| | | | | | | | | prologue, then loads & stores x20 on the stack from a different location in the middle of the function, and then restores the reg in the epilogue. The saving/restoring of x20 in the middle of the function should be ignored. llvm-svn: 283969
* Add a simple frameless function test case to the arm64 unwind unit testsJason Molenda2016-10-122-11/+107
| | | | | | with return statements in the body of the function. llvm-svn: 283966
* This test now passes.Jim Ingham2016-10-121-2/+0
| | | | llvm-svn: 283959
* This was skipped due to a clang PR that has been fixed.Jim Ingham2016-10-111-1/+0
| | | | llvm-svn: 283957
* Added a radar on our end for this test's failure.Jim Ingham2016-10-111-1/+1
| | | | llvm-svn: 283956
* Add the radar number on our end.Jim Ingham2016-10-111-1/+1
| | | | llvm-svn: 283940
* This test now passes.Jim Ingham2016-10-111-1/+0
| | | | llvm-svn: 283929
* Added a bugreport tracking the failure to get float return valuesJim Ingham2016-10-111-5/+6
| | | | | | on i386. llvm-svn: 283923
* Clarified the explanation of expr --top-level.Sean Callanan2016-10-111-2/+2
| | | | llvm-svn: 283904
* Fix Arm64InstEmulation tests for windowsPavel Labath2016-10-111-15/+4
| | | | | | | | MSVC does not like the declaration of a terminate() function (I guess it looks too much like std::terminate()). While I'm there, move the setup/teardown code into the functions gtest provides for that purpose. llvm-svn: 283870
* Add a second, more complicated, arm64 example program toJason Molenda2016-10-111-11/+169
| | | | | | the arm64 assembly unwind tests. llvm-svn: 283849
* Add a first unit test for the arm64 instruction profiled unwindJason Molenda2016-10-118-17/+241
| | | | | | | | | | | | | | | | | | | | | | | plan generator. Fix a small bug in EmulateInstructionARM64::GetFramePointerRegister which was returning the stack pointer reg instead of fp, prevented the unwinder from recognizing the switch to using the fp in a function. (<rdar://problem/28663117>) Add a new eContextRestoreStackPointer context hint so that the arm64 emulator can flag when the frame pointer value is copied back in to the stack pointer and that should be used to compute the canonical frame address again in an epilogue sequence. (<rdar://problem/28704862>) Small changes to UnwindAssemblyInstEmulation to have a method we can call without a live process/thread/etc for unit tests. <rdar://problem/28663117> <rdar://problem/28704862> <rdar://problem/28509178> llvm-svn: 283847
* Fixing the bug number for darwin on this failure.Jim Ingham2016-10-111-1/+3
| | | | llvm-svn: 283843
* This test was failing because /bin/ls is no longer debuggable on OS X.Jim Ingham2016-10-113-2/+19
| | | | | | Add an executable that we can debug. llvm-svn: 283835
* Adding radar number on our end for llvm.org/pr15824.Jim Ingham2016-10-101-2/+2
| | | | llvm-svn: 283821
* Add the related radar on our end for llvm.org/pr15824.Jim Ingham2016-10-101-1/+1
| | | | llvm-svn: 283820
* Add the radar number on our end.Jim Ingham2016-10-101-1/+1
| | | | llvm-svn: 283813
* [LLDB][MIPS] All tests get errors in dotest after this test.Nitesh Jain2016-10-102-0/+6
| | | | | Subscribers: jaydeep, bhushan, slthakur, llvm-commits llvm-svn: 283739
* [LLDB][MIPS] Skip some test case which were causing LLDB to go into infinite ↵Nitesh Jain2016-10-1035-35/+35
| | | | | | | | | | | | loop Reviewers: clayborg, labath Subscribers: jaydeep, bhushan, slthakur, llvm-commits Differential Revision: https://reviews.llvm.org/D24549 llvm-svn: 283732
* [LLDB][MIPS] Fix TestReturnValue failure for MIPSNitesh Jain2016-10-101-27/+39
| | | | | | | | | | Reviewers: clayborg, labath, bhushan Subscribers: jaydeep, slthakur, llvm-commits Differential Revision: https://reviews.llvm.org/D24498 llvm-svn: 283729
* [LLDB][MIPS] Fix register read/write for 32 bit big endian systemNitesh Jain2016-10-102-1/+4
| | | | | | | | | | Reviewers: clayborg, labath Subscribers: jaydeep, bhushan, mohit.bhakkad, slthakur, llvm-commits Differential Revision: https://reviews.llvm.org/D24124 llvm-svn: 283728
* Fix compiler warnings in TestClangASTContext.cppPavel Labath2016-10-081-6/+6
| | | | llvm-svn: 283652
* Fix compiler warnings in PlatformDarwinTest.cppPavel Labath2016-10-081-16/+16
| | | | llvm-svn: 283651
* Fix a few warnings caught by clang.Zachary Turner2016-10-072-2/+2
| | | | llvm-svn: 283607
* Remove a stray dump().Sean Callanan2016-10-071-2/+0
| | | | | | <rdar://problem/28635530> llvm-svn: 283604
OpenPOWER on IntegriCloud