[PATCH] DW_OP_const* is invalid as a location

Katya Romanova Katya_Romanova at playstation.sony.com
Mon Dec 8 11:53:21 PST 2014


This is a fix for BZ #21776.
http://llvm.org/bugs/show_bug.cgi?id=21776

Before DW_OP_stack_value was introduced in DWARF4, llvm relied on heuristics to disambiguate the value vs. location status of the expression DW_OP_consts <const>.

The proper (compliant with DWARF) way to describe a location where a value is constant is "DW_OP_constu <const>, DW_OP_stack_valueā€¯.

The fix is simple. I pushed DW_OP_stack_value on the stack.



, DW_OP_consts <const> supposed to represent

http://reviews.llvm.org/D6568

Files:
  lib/CodeGen/AsmPrinter/DwarfDebug.cpp
  test/DebugInfo/incorrect-variable-debugloc1.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6568.17045.patch
Type: text/x-patch
Size: 6283 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141208/49512e43/attachment.bin>


More information about the llvm-commits mailing list