Re: [PATCH] Fix -fdump-rtl-sms (PR rtl-optimization/52095)

2012-02-06 Thread Richard Guenther
On Fri, Feb 3, 2012 at 7:07 PM, Jakub Jelinek wrote: > Hi! > > On some targets e.g. sms-7.c test fails, because fprintf is called > with %s format and NULL argument, GLIBC prints for that e.g. > SMS loop num: 1, file: (null), line: 0 > but it isn't portable.  print-rtl.c guards the locator printin

Re: [PATCH] Fix -fdump-rtl-sms (PR rtl-optimization/52095)

2012-02-04 Thread Ayal Zaks
On Fri, Feb 3, 2012 at 8:07 PM, Jakub Jelinek wrote: > Hi! > > On some targets e.g. sms-7.c test fails, because fprintf is called > with %s format and NULL argument, GLIBC prints for that e.g. > SMS loop num: 1, file: (null), line: 0 > but it isn't portable.  print-rtl.c guards the locator printin

[PATCH] Fix -fdump-rtl-sms (PR rtl-optimization/52095)

2012-02-03 Thread Jakub Jelinek
Hi! On some targets e.g. sms-7.c test fails, because fprintf is called with %s format and NULL argument, GLIBC prints for that e.g. SMS loop num: 1, file: (null), line: 0 but it isn't portable. print-rtl.c guards the locator printing with /* Pretty-print insn locators. Ignore scopin