From: Bernhard H. <ber...@be...> - 2002-08-21 12:52:16
|
> > I wouldn't call it a "fix". It's a workaround to disable parts of the > > optimizer. > > Yes it is a workaround! It does fix the printf problem at fairly > minimal cost in terms of code size. It obviously would be nice to fix > the optimiser bug too... > > > Most likely bug #597690 is the same as #536787. I'll close the bug > > report. > > So will you patch vprintf in the CVS archive with a similar > workaround? I don't want to change vprintf in CVS, because it's a workaround, not a fix. IMHO it's not a good idea to polute the source with workarounds, unless there are very, very good reasons to do so. Neither vprinf.c nor sdcc is my work or my software. I refuse to mess around with other peoples work. I won't change anything, unless I'm absolutely sure, that I'm doing the right thing. And I'm not sure, if workarounds are right way to go. Bernhard |