[PATCH] D51075: [SafeStack] Set debug location for calls to __safestack_pointer_address.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 21 16:53:48 PDT 2018


efriedma created this revision.
efriedma added a reviewer: eugenis.
Herald added subscribers: JDevlieghere, kristof.beyls, aprantl, srhines.
Herald added a reviewer: javed.absar.

Otherwise, the debug info is incorrect.  On its own, this is mostly harmless, but safe-stack also later inlines the call to __safestack_pointer_address, which leads to debug info with the wrong scope, which eventually causes an assertion failure (and incorrect debug info in release mode).


Repository:
  rL LLVM

https://reviews.llvm.org/D51075

Files:
  lib/CodeGen/SafeStack.cpp
  test/Transforms/SafeStack/ARM/debug.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51075.161849.patch
Type: text/x-patch
Size: 6664 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180821/ab457a8e/attachment.bin>


More information about the llvm-commits mailing list