[PATCH] D58726: [DebugInfo][Docs] Explicitly document how dbg.value intrinsics are interpreted in optimized code

[Vedant Kumar via Phabricator via llvm-commits]

vsk added a comment.

+1, thanks for codifying all of this!



================
Comment at: docs/SourceLevelDebugging.rst:403
+``llvm.dbg.value`` intrinsic is created for each assignment, recording the
+variables new location. Compared with the ``llvm.dbg.declare`` intrinsic:
+
----------------
variable's


================
Comment at: docs/SourceLevelDebugging.rst:413
+Recording this debugging information exposes it to optimization transformations
+to a much greater extent. For example:
+
----------------
Can this sentence ('Recording ... to a much greater extent') be a bit more direct? It might work to simply move it after the next paragraph, saying e.g. 'Care must be taken to update dbg.value intrinsics when: ...'.


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

https://reviews.llvm.org/D58726