<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 20, 2013 at 12:57 PM, Duncan Sands <span dir="ltr"><<a href="mailto:baldrick@free.fr" target="_blank">baldrick@free.fr</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi David,<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
    - some Fortran testcases crash, but it's not your fault: it occurs with<br>
    pointers to arrays, due to hitting this:<br>
<br>
       if (isa<ARRAY_TYPE>(type) && TYPE_STRING_FLAG(type) &&<br></div>
           isa<INTEGER_TYPE>(TREE_TYPE(__<u></u>type))) {<div class="im"><br>
         DEBUGASSERT(0 && "Don't support pascal strings");<br>
         return DIType();<br>
       }<br>
<br>
    Then the empty DIType is used by the pointer code to try and form a<br>
    derived type.  I'm not sure why this (ancient) debug code thinks there<br>
    is a problem with this kind of array, so I will try to fix it to handle<br>
    them properly.<br>
</div></blockquote>
<br>
I've "fixed" this by removing this code and letting "pascal strings" fall<br>
through to the general array case.  I think this is good enough for the moment.<br></blockquote><div><br></div><div style>Thanks</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Your patch still applies, and now the only failure is the objc member problem<br>
I mentioned before.</blockquote><div><br></div><div style>Looking at this, it seems like there's no change in behavior - the test appears to have been accidentally passing because the IR dumper wasn't printing out the annotated debug info comments next to the metadata (I guess the IR printer lost support for annotating the old debug info format somewhere along the way)<br>
<br>Attached are the old (ToT) & new (with my patch applied) IR for this test case - perhaps I'm missing something about what the test case is meant to test, etc.<br><br>- David</div></div></div></div>