[PATCH] D13178: Fix SafeStack debug locations

[Alexey Samsonov via llvm-commits]

samsonov added a comment.

Nice!


================
Comment at: include/llvm/Transforms/Utils/Local.h:276
@@ +275,3 @@
+/// a new value. If Deref is true, an additional DW_OP_deref is prepended to the
+/// expression. If Offset is non-zero, a constant displacement is added to the
+/// expression (after the optional Deref). Offset can be negative.
----------------
Please elaborate what's actually going on - whether you rewrite expr to be (*orig + offset), or *(orig + offset) - it's not entirely clear from comment

================
Comment at: lib/IR/DebugInfoMetadata.cpp:519
@@ -517,2 +518,3 @@
       return I->get() + I->getSize() == E->get();
     case dwarf::DW_OP_plus:
+    case dwarf::DW_OP_minus:
----------------
This looks weird to me: why don't we use std::advance(I, I->getSize()) instead?


Repository:
  rL LLVM

http://reviews.llvm.org/D13178