I added a test that illustrates what @vsk is talking about.
May 25 2018
May 24 2018
Can we step back a second and better explain what the problem is? With current Clang the debug info for this function looks okay to me.
The store that is "missing" a debug location is homing the formal parameter to its local stack location; this is part of prolog setup, not "real" code.
Adress review comments.
Limit changes to the storeInst
May 23 2018
Remove redundant this
Set debug location to the entry block alloca
It was a mistake on my part with the arcanist tool. The documentation is fine, the only problems I encountered were due to my ignorance of how these systems work.
May 22 2018
Move ApplyDebugLocation before CreateMemTemp
May 21 2018
Update according to the comments
- Apply debug location
May 18 2018
May 17 2018
fix mistake in tests
The output of the tests should go to /dev/null
The tests were missing the input file argument
Test that debugify can be safely used with piping
- [Debugify] All output now goes to stderr
May 16 2018
Just like llvm-lit I left the PASS or FAIL to stdout.
May 15 2018
Will do, it's allready in my TODO list!
- Adress review comments
Mar 18 2018
Mar 13 2018
- Fix test label
- fix redundant debugify test