diff options
| author | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2015-07-24 23:59:25 +0000 |
|---|---|---|
| committer | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2015-07-24 23:59:25 +0000 |
| commit | 56b893b364eba35f4d64fc92251de140ad5178ea (patch) | |
| tree | 82fea6f3317b29bbff985580981990c5a14abdb8 /llvm/examples/Kaleidoscope/Chapter8 | |
| parent | cd3f99b6c534ad0d0e2040c6beea8e82b575123b (diff) | |
| download | bcm5719-llvm-56b893b364eba35f4d64fc92251de140ad5178ea.tar.gz bcm5719-llvm-56b893b364eba35f4d64fc92251de140ad5178ea.zip | |
DI/Verifier: Fix argument bitrot in DILocalVariable
Add a verifier check that `DILocalVariable`s of tag
`DW_TAG_arg_variable` always have a non-zero 'arg:' field, and those of
tag `DW_TAG_auto_variable` always have a zero 'arg:' field. These are
the only configurations that are properly understood by the backend.
(Also, fix the bad examples in LangRef and test/Assembler, and fix the
bug in Kaleidoscope Ch8.)
A large number of testcases seem to have bitrotted their way forward
from some ancient version of the debug info hierarchy that didn't have
`arg:` parameters. If you have out-of-tree testcases that start failing
in the verifier and you don't care enough to get the `arg:` right, you
may have some luck just calling:
sed -e 's/, arg: 0/, arg: 1/'
or some such, but I hand-updated the ones in tree.
llvm-svn: 243183
Diffstat (limited to 'llvm/examples/Kaleidoscope/Chapter8')
| -rw-r--r-- | llvm/examples/Kaleidoscope/Chapter8/toy.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/examples/Kaleidoscope/Chapter8/toy.cpp b/llvm/examples/Kaleidoscope/Chapter8/toy.cpp index dbc996d4318..72e5089177b 100644 --- a/llvm/examples/Kaleidoscope/Chapter8/toy.cpp +++ b/llvm/examples/Kaleidoscope/Chapter8/toy.cpp @@ -1255,7 +1255,7 @@ void PrototypeAST::CreateArgumentAllocas(Function *F) { KSDbgInfo.TheCU->getDirectory()); DILocalVariable *D = DBuilder->createLocalVariable( dwarf::DW_TAG_arg_variable, Scope, Args[Idx], Unit, Line, - KSDbgInfo.getDoubleTy(), Idx); + KSDbgInfo.getDoubleTy(), true, 0, Idx + 1); DBuilder->insertDeclare(Alloca, D, DBuilder->createExpression(), DebugLoc::get(Line, 0, Scope), |

