Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | [lldb][NFC] Remove all `setUp` overrides that only call the parent ↵ | Raphael Isemann | 2019-12-13 | 1 | -3/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | | implementation Summary: A lot of our tests copied the setUp code from our TestSampleTest.py: ``` def setUp(self): # Call super's setUp(). TestBase.setUp(self) ``` This code does nothing unless we actually do any setUp work in there, so let's remove all these method definitions. Reviewers: labath, JDevlieghere Reviewed By: labath Subscribers: lldb-commits Tags: #lldb Differential Revision: https://reviews.llvm.org/D71454 | ||||
* | [lldb][NFC] Remove 'from __future__ import print_function' from all tests ↵ | Raphael Isemann | 2019-12-13 | 1 | -1/+0 |
| | | | | | | | | | | | | | | | | | | | | | that don't actually call 'print()' Summary: A lot of tests do this trick but the vast majority of them don't even call `print()`. Most of this patch was generated by a script that just looks at all the files and deletes the line if there is no `print (` or `print(` anywhere else in the file. I checked the remaining tests manually and deleted the import if we never call print (but instead do stuff like `expr print(...)` and similar false-positives). I also corrected the additional empty lines after the import in the files that I manually edited. Reviewers: JDevlieghere, labath, jfb Reviewed By: labath Subscribers: dexonsmith, wuzish, nemanjai, kbarton, christof, arphaman, abidh, lldb-commits Tags: #lldb Differential Revision: https://reviews.llvm.org/D71452 | ||||
* | [-gmodules] Let LLDB log a warning if the Clang module hash mismatches. | Adrian Prantl | 2019-11-15 | 4 | -0/+75 |
This feature is mostly there to aid debugging of Clang module issues, since the only useful actual the end-user can to is to recompile their program. Differential Revision: https://reviews.llvm.org/D70272 |