On 08/25/2016 10:23 AM, David Malcolm wrote:
Martin: here are the fixups for your patch I needed to apply to make
it work with mine. I couldn't actually get any of your existing test
cases to emit locations within the string literals, due to them all
being embedded in macro expansions (possibly relating to PR c/77328),
so I added a simple testcase using -fdiagnostics-show-caret, which
does successfully show a range within the string.
Posting in the hope that it's helpful; I haven't attempted a bootstrap
with it.
Thanks! Let me go through it, test it and get back to you when I'm
done (sometime next week after I get back from PTO).
Martin
