On Thu, Jul 9, 2015 at 5:23 PM, Michael Matz m...@suse.de wrote:
Hi,
On Thu, 9 Jul 2015, Jakub Jelinek wrote:
That violates the coding style by not using tabs ;)
I knew it! Somebody would notice, pffft. Fixed in the committed version.
I also noticed it but didn't care ;) But now I
On Fri, Jul 10, 2015 at 10:45 AM, Richard Biener
richard.guent...@gmail.com wrote:
On Thu, Jul 9, 2015 at 5:23 PM, Michael Matz m...@suse.de wrote:
Hi,
On Thu, 9 Jul 2015, Jakub Jelinek wrote:
That violates the coding style by not using tabs ;)
I knew it! Somebody would notice, pffft.
Hi,
On Fri, 10 Jul 2015, Richard Biener wrote:
I also noticed it but didn't care ;) But now I notice
switch (TREE_CODE (t))
{
case SSA_NAME:
cases are indented too much, it should be
switch (TREE_CODE (t))
{
case SSA_NAME:
I like the first one
On 07/09/2015 06:20 AM, Michael Matz wrote:
Hi,
while looking at gimple-match.c I got a minor stroke, so this patch makes
genmatch generated (mostly) properly indented code. Sure, it could be
done post-fact by an editor or something when one looks at the file, but
all other generators also try
On Thu, Jul 09, 2015 at 02:20:08PM +0200, Michael Matz wrote:
+/* Like fprintf, but print INDENT spaces at the beginning. */
+
+static void
+#if GCC_VERSION = 4001
+__attribute__((format (printf, 3, 4)))
+#endif
+fprintf_indent (FILE *f, unsigned int indent, const char *format, ...)
+{
+
Hi,
On Thu, 9 Jul 2015, Jakub Jelinek wrote:
That violates the coding style by not using tabs ;)
I knew it! Somebody would notice, pffft. Fixed in the committed version.
Ciao,
Michael.
PS: this still isn't fully correct, as sometimes I start the strings with
spaces which don't count
On Thu, Jul 9, 2015 at 2:20 PM, Michael Matz m...@suse.de wrote:
Hi,
while looking at gimple-match.c I got a minor stroke, so this patch makes
genmatch generated (mostly) properly indented code. Sure, it could be
done post-fact by an editor or something when one looks at the file, but
all
Hi,
while looking at gimple-match.c I got a minor stroke, so this patch makes
genmatch generated (mostly) properly indented code. Sure, it could be
done post-fact by an editor or something when one looks at the file, but
all other generators also try to generate good looking code. No
