[PATCH] D143567: [Assignment Tracking] Fix fragment index error in getDerefOffsetInBytes

Orlando Cazalet-Hyams via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 8 02:59:57 PST 2023


Orlando created this revision.
Orlando added reviewers: StephenTozer, jmorse, scott.linder.
Orlando added a project: debug-info.
Herald added subscribers: arphaman, hiraditya.
Herald added a project: All.
Orlando requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Without this patch `getDerefOffsetInBytes` incorrectly always returns `std::nullopt` for expressions with fragments due to an off-by-one error with fragment element indices.


https://reviews.llvm.org/D143567

Files:
  llvm/lib/CodeGen/AssignmentTrackingAnalysis.cpp
  llvm/test/DebugInfo/assignment-tracking/X86/lower-to-value.ll
  llvm/test/DebugInfo/assignment-tracking/X86/mem-loc-frag-fill.ll
  llvm/test/DebugInfo/assignment-tracking/X86/nested-loop-frags.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D143567.495782.patch
Type: text/x-patch
Size: 7094 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230208/cbe0a516/attachment.bin>


More information about the llvm-commits mailing list