[PATCH] D35951: Remove the obsolete "offset" parameter from @llvm.dbg.value
Adrian Prantl via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 27 21:45:14 PDT 2017
aprantl added a comment.
In https://reviews.llvm.org/D35951#823564, @dexonsmith wrote:
> The code changes and test/Bitcode/upgrade-dbg-value.ll LGTM.
>
> I spot-checked the other testcases, and couldn't find any that had anything other than `i64 0`. Is that expected? Or can you point me at a non-trivial one?
We have been phasing out any remaining use of the offset field and replaced it with DIExpressions. This is also why the upgrade merely strips any dbg.value intrinsics with nonzero offsets. It's effectively dead code.
Repository:
rL LLVM
https://reviews.llvm.org/D35951
More information about the llvm-commits
mailing list