Re: [PATCH] Fixups for Martin's gimple-ssa-sprintf.c patch

2016-08-31 Thread Martin Sebor
On 08/31/2016 10:26 AM, David Malcolm wrote: On Wed, 2016-08-31 at 10:23 -0600, Martin Sebor wrote: On 08/25/2016 10:30 AM, Martin Sebor wrote: 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 actua

Re: [PATCH] Fixups for Martin's gimple-ssa-sprintf.c patch

2016-08-31 Thread Martin Sebor
On 08/25/2016 10:30 AM, Martin Sebor wrote: 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

Re: [PATCH] Fixups for Martin's gimple-ssa-sprintf.c patch

2016-08-31 Thread David Malcolm
On Wed, 2016-08-31 at 10:23 -0600, Martin Sebor wrote: > On 08/25/2016 10:30 AM, Martin Sebor wrote: > > 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 existi

Re: [PATCH] Fixups for Martin's gimple-ssa-sprintf.c patch

2016-08-25 Thread Martin Sebor
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