[PATCH] D64971: [SafeStack] Insert the deref after the offset

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 19 13:08:33 PDT 2019


vsk accepted this revision as: vsk.
vsk added a comment.
This revision is now accepted and ready to land.

In D64971#1593932 <https://reviews.llvm.org/D64971#1593932>, @phosek wrote:

> I've moved the `deref` after the offset calculation, tested this against our build and it's producing the expected DWARF output.


Thanks, LGTM. @jmorse any outstanding concerns?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64971/new/

https://reviews.llvm.org/D64971





More information about the llvm-commits mailing list