<div dir="ltr">I believe duncan's fixed this recently in <span style="font-family:monospace">235956 and </span><span style="font-family:monospace">235955 - does ToT work for you?<br></span><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 28, 2015 at 7:07 PM, Sandeep Raju <span dir="ltr"><<a href="mailto:srand48@gmail.com" target="_blank">srand48@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi Folks,<br>
<br>
I ran into this assertion failure while compiling a function with a<br>
large number of arguments:<br>
<br>
<a href="https://llvm.org/bugs/show_bug.cgi?id=21609" target="_blank">https://llvm.org/bugs/show_bug.cgi?id=21609</a><br>
<br>
I have coded up the fix as per David's suggestion (added a new header<br>
field for DIVariable to separate out ArgNo & LineNo). The proposed<br>
diff is attached to the bug.<br>
<br>
However, there are around 175 testcases across clang & llvm that need<br>
to be udpated to reflect the new schema. One such eg:<br></blockquote><div><br>The debug info has changed substantially in the last few months - those changes made a lot of churn to the test cases and, as a biproduct, made it unnecessary to update any test cases for this change.<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
diff --git a/test/Instrumentation/AddressSanitizer/debug_info.ll<br>
b/test/Instrumentation/AddressSanitizer/debug_info.ll<br>
index c0939c5..4112f14 100644<br>
--- a/test/Instrumentation/AddressSanitizer/debug_info.ll<br>
+++ b/test/Instrumentation/AddressSanitizer/debug_info.ll<br>
@@ -41,9 +41,9 @@ declare void @llvm.dbg.declare(metadata, metadata,<br>
metadata) nounwind readnone<br>
 !7 = !{!"0x15\00\000\000\000\000\000\000", i32 0, null, null, !8,<br>
null, null, null} ; [ DW_TAG_subroutine_type ] [line 0, size 0, align<br>
0, offset 0] [from ]<br>
 !8 = !{!9, !9}<br>
 !9 = !{!"0x24\00int\000\0032\0032\000\000\005", null, null} ; [<br>
DW_TAG_base_type ] [int] [line 0, size 32, align 32, offset 0, enc<br>
DW_ATE_signed]<br>
-!10 = !{!"0x101\00p\0016777217\000", !5, !6, !9} ; [<br>
DW_TAG_arg_variable ] [p] [line 1]<br>
+!10 = !{!"0x101\00p\001\001\000", !5, !6, !9} ; [ DW_TAG_arg_variable<br>
] [p] [line 1]<br>
 !11 = !MDLocation(line: 1, scope: !5)<br>
-!12 = !{!"0x100\00r\002\000", !13, !6, !9} ; [ DW_TAG_auto_variable ]<br>
[r] [line 2]<br>
+!12 = !{!"0x100\00r\002\000\000", !13, !6, !9} ; [<br>
DW_TAG_auto_variable ] [r] [line 2]<br>
<br>
Before I spend cycles updating the test cases, I wanted get a 'go<br>
ahead' from code owners.<br>
<br>
Please let me know.<br>
<br>
Thanks,<br>
Sandeep<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu" target="_blank">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</blockquote></div><br></div></div>