[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