[PATCH] DW_OP_const* is invalid as a location

Katya Romanova Katya_Romanova at playstation.sony.com
Thu Dec 11 13:38:21 PST 2014


Buildbots spotted a failure for PowerPC platform for the incorrect-variable-debugloc1.ll test that I added. 
Uploading a diff once again.

(1) Eric asked me to add a comment with the explanation in the source file (for the patch itself). Done.
(2) David asked me to change BZ 21776 into PR21776 in the testcase's comment. Done.
(3) I added XFAIL condition (and a comment) for PowerPC platform.

Does it look good?


http://reviews.llvm.org/D6568

Files:
  lib/CodeGen/AsmPrinter/DwarfDebug.cpp
  test/DebugInfo/incorrect-variable-debugloc1.ll

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6568.17188.patch
Type: text/x-patch
Size: 5855 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141211/f66ea5c7/attachment.bin>


More information about the llvm-commits mailing list