summaryrefslogtreecommitdiffstats
path: root/debuginfo-tests/nrvo-string.cpp
Commit message (Collapse)AuthorAgeFilesLines
* Reapply "Import Dexter to debuginfo-tests""Jeremy Morse2019-10-311-52/+0
| | | | | | | This reverts commit cb935f345683194e42e6e883d79c5a16479acd74. Discussion in D68708 advises that green dragon is being briskly refurbished, and it's good to have this patch up testing it.
* Revert "Import Dexter to debuginfo-tests"Jeremy Morse2019-10-311-0/+52
| | | | | | This reverts commit f78c236efda85af1e526ac35ed535ef4786450e3. Green dragon breakage was observed; I'll take a look at why.
* Import Dexter to debuginfo-testsJeremy Morse2019-10-311-52/+0
| | | | | | | | | | | | | | | | | Dexter (Debug Experience Tester) is a test-driver for our debug info integration tests, reading a set of debug experience expectations and comparing them with the actual behaviour of a program under a debugger. More about Dexter can be found in the RFC: http://lists.llvm.org/pipermail/llvm-dev/2019-October/135773.html and the phab review in D68708. Not all the debuginfo tests have been transformed into Dexter tests, and we look forwards to doing that incrementally. This commit mostly aims to flush out buildbots that are running debuginfo-tests but don't have python 3 installed, possibly green-dragon and some windows bots.
* Make nrvo-string test more robust.Adrian Prantl2019-06-271-3/+4
| | | | | | | | | | | | | | | | This is a follow-up to r364466, but better implemented. Original commit message still applies: The breakpoint locations were in places where clang doesn't actually emit a source location for and depend on the debugger's ability to move the breakpoint forward onto a line that is already in the function epilogue. In my testing older versions of LLDB fail to do that, so I'm modifying the test to force a break-able location by calling a noinline function. <rdar://problem/52079841> llvm-svn: 364589
* Make nrvo-string test more robust.Adrian Prantl2019-06-261-2/+5
| | | | | | | | | | | | | The breakpoint locations were in places where clang doesn't actually emit a source location for and depend on the debugger's ability to move the breakpoint forward onto a line that is already in the function epilogue. In my testing older versions of LLDB fail to do that, so I'm modifying the test to force a break-able location by calling a noinline function. <rdar://problem/52079841> llvm-svn: 364466
* Store a pointer to the return value in a static alloca and let the debugger ↵Amy Huang2019-06-201-2/+23
| | | | | | | | | | | | | | use that as the variable address for NRVO variables. Subscribers: hiraditya, cfe-commits, llvm-commits Tags: #clang, #llvm Differential Revision: https://reviews.llvm.org/D63361 llvm-svn: 363952
* Revert "Temporarily pin tests to DWARF v2 until a more recent version of LLDB"Adrian Prantl2017-12-071-2/+2
| | | | | | | | This reverts commit 319790. We worked around the bug in LLVM instead. llvm-svn: 320076
* Temporarily pin tests to DWARF v2 until a more recent version of LLDBAdrian Prantl2017-12-051-2/+2
| | | | | | is available on green dragon. llvm-svn: 319790
* Re-revert "Refactor debuginfo-tests."Zachary Turner2017-11-211-0/+27
| | | | | | | | | | This is still breaking greendragon. At this point I give up until someone can fix the greendragon bots, and I will probably abandon this effort in favor of using a private github repository. llvm-svn: 318722
* Resubmit "Refactor debuginfo-tests" again.Zachary Turner2017-11-201-27/+0
| | | | | | | | | | | | This was reverted due to the tests being run twice on some build bots. Each run had a slightly different configuration due to the way in which it was being invoked. This fixes the problem (albeit in a somewhat hacky way). Hopefully in the future we can get rid of the workflow of running debuginfo-tests as part of clang, and then this hack can go away. llvm-svn: 318697
* Re-revert "Refactor debuginfo-tests"Zachary Turner2017-11-171-0/+27
| | | | | | | | | | | | | | | | | | | | | | This is still broken because it causes certain tests to be run twice with slightly different configurations, which is wrong in some cases. You can observe this by running: ninja -nv check-all | grep debuginfo-tests And seeing that it passes clang/test and clang/test/debuginfo-tests to lit, which causes it to run debuginfo-tests twice. The fix is going to involve either: a) figuring out that we're running in this "deprecated" configuration, and then deleting the clang/test/debuginfo-tests path, which should cause it to behave identically to before, or: b) make lit smart enough that it doesn't descend into a sub-suite if that sub-suite already has a lit.cfg file. llvm-svn: 318486
* Resubmit "Refactor debuginfo-tests"Zachary Turner2017-11-161-27/+0
| | | | | | | | | | This was reverted due to some failures on specific darwin buildbots, the issue being that the new lit configuration was not setting the SDKROOT environment variable. We've tested a fix locally and confirmed that it works, so this patch resubmits everything with the fix applied. llvm-svn: 318435
* Revert "Update test_debuginfo.pl script to point to new tree location."Zachary Turner2017-11-131-0/+27
| | | | | | | | This reverts the aforementioned patch and 2 subsequent follow-ups, as some buildbots are still failing 2 tests because of it. Investigation is ongoing into the cause of the failures. llvm-svn: 318112
* [debuginfo-tests] Make debuginfo-tests work in a standard configuration.Zachary Turner2017-11-101-27/+0
| | | | | | | | | | | | | | | | | | | Previously, debuginfo-tests was expected to be checked out into clang/test and then the tests would automatically run as part of check-clang. This is not a standard workflow for handling external projects, and it brings with it some serious drawbacks such as the inability to depend on things other than clang, which we will need going forward. The goal of this patch is to migrate towards a more standard workflow. To ease the transition for build bot maintainers, this patch tries not to break the existing workflow, but instead simply deprecate it to give maintainers a chance to update the build infrastructure. Differential Revision: https://reviews.llvm.org/D39605 llvm-svn: 317925
* Revert r313600 due to bot failures on Green Dragon.Mike Edwards2017-09-191-0/+2
| | | | | | http://green.lab.llvm.org/green/job/clang-stage1-configure-RA_check/35585/ llvm-svn: 313634
* Drop -O1 case from nrvo-string.cppHans Wennborg2017-09-181-2/+0
| | | | | | It relied on r313400 which was reverted in r313589. llvm-svn: 313600
* [debuginfo-tests] Add string NRVO test for PR34513Reid Kleckner2017-09-151-0/+27
It should pass in -O0 and -O1 after r313400 and r313399. llvm-svn: 313401
OpenPOWER on IntegriCloud