<div dir="ltr"><div><div><div>Hi Aaron,<br><br></div>Sorry.  I have uploaded the test case to the Phabricator.  Now I attached the test case in the email.<br><br></div>I have tried to trigger the issue with -ast-dump (or something similar) but failed.  It seems that getAttrEnumOperandLoc() is only used by ASTReader, ASTWriter, and TransGCAttrs.  None of them will try to print the source location to the output.  Do you have any idea?  Thanks.<br>
<br></div><div>Sincerely,<br></div>Logan<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Mar 10, 2014 at 12:01 AM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron.ballman@gmail.com" target="_blank">aaron.ballman@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I don't see anything attached to your email, but I don't think crashes<br>
aren't required for a test case for this. The behavior I was thinking<br>
of testing is that the old code would fail to set the operand location<br>
properly for the type attribute, and the new case does set it. So any<br>
case which relies on that location being set to something valid would<br>
suffice.<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Sun, Mar 9, 2014 at 11:55 AM, Logan Chien <<a href="mailto:tzuhsiang.chien@gmail.com">tzuhsiang.chien@gmail.com</a>> wrote:<br>
><br>
>   If you are interested in, here's reduced test case.  It was reduced from the Android jni.h.  I feel this test is not good enough because clang will not crash if you delete some of the lines in crash_pch.h.<br>

><br>
>   {F47242}<br>
><br>
> <a href="http://llvm-reviews.chandlerc.com/D3020" target="_blank">http://llvm-reviews.chandlerc.com/D3020</a><br>
</div></div></blockquote></div><br></div>