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/docs/tutorial | |
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/docs/tutorial')
-rw-r--r-- | llvm/docs/tutorial/LangImpl8.rst | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/docs/tutorial/LangImpl8.rst b/llvm/docs/tutorial/LangImpl8.rst index 0b9b39c84b7..88224bb92f0 100644 --- a/llvm/docs/tutorial/LangImpl8.rst +++ b/llvm/docs/tutorial/LangImpl8.rst @@ -397,7 +397,7 @@ argument allocas in ``PrototypeAST::CreateArgumentAllocas``. 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); Instruction *Call = DBuilder->insertDeclare( Alloca, D, DBuilder->createExpression(), Builder.GetInsertBlock()); |