<div dir="ltr">What was the exact output?<div><br></div><div>I would expect the source location reported for the write and then just the message with the line/column info but without</div><div>source code.</div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Thu, Jun 26, 2014 at 7:13 PM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="">On 26/06/2014 16:14, Tobias Grosser wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 26/06/2014 16:08, Andreas Simbuerger wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<br>
<br>
<br>
On 06/26/2014 04:00 PM, Tobias Grosser wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 26/06/2014 15:38, Andreas Simbuerger wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1<br>
<br>
At the moment the DebugLoc is derived from the BaseValue. I'm not<br>
100% sure if the resulting DebugLoc corresponds to the A or the<br>
(A->b) BaseValue.<br>
</blockquote>
<br>
Possibly A->b.<br>
</blockquote>
<br>
Yep, I meant 100%-sure from the output of the DebugLoc ;-).<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
As far as I can tell, both would map to the same source location<br>
in the debug-metadata.<br>
</blockquote>
<br>
The caret diagnostic points for me at the beginning of the line.<br>
This is good. If we ever figure out how to get the precise column<br>
location, that would be amazing.<br>
</blockquote>
<br>
That's what I would love to have. But well, so far it should be<br>
helpful enough. ;-)<br>
</blockquote>
<br>
Its extremely useful already. No complains from my side. Just putting<br>
this on the 'nice-to-have-if-someone-has-<u></u>an-idea-and-time' list.<br>
</blockquote>
<br></div>
Diego's thread just pointed me to -gcolumn-info. After enabling it we already get the caret at the right location. So we only need to wait for it being enabled.<br>
<br>
I also tested the following SCoP with -polly-detect-keep-going:<br>
<br>
 void a(struct b *A) {<div class=""><br>
   for (int i=0; i<32; i++)<br></div>
     A->b[i] = A->b[i+1];<br>
 }<br>
<br>
Only the first location comes with a source location. I wonder if we could fix this by looking at the location of the load instruction, as it is less likely to be removed by gvn?<br>
<br>
Cheers,<br>
Tobias<br>
<br>
</blockquote></div><br></div>