This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [Patch, Fortran] Experimental LEN_TRIM and TRIM patch


Jerry DeLisle wrote:
Daniel Kraft wrote:
Slightly improved patch and ChangeLog.

This is OK for 4.5. I think the case of short strings is not very likely. I suppose even for that we could special case it.

Thanks, but I of course still need to figure out how to check for 64-bit systems with the preprocessor :)


We could of course do a special case, but the problem is not about overall string length but rather *few trailing blanks*. So as I too believe longer strings are more often encountered (and of course for short strings both versions are quite fast), I think we can leave it like this.

I will update the Wiki's patch-tracker once I've got the final patch with correct conditional compilation.

BTW, no regressions on GNU/Linux-x86-32. But It would be cool to test this on other systems (and 64-bit ones) as well once the final patch is there.

Daniel

--
Done:  Arc-Bar-Cav-Ran-Rog-Sam-Tou-Val-Wiz
To go: Hea-Kni-Mon-Pri


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]