[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