Hi Paul, Paul Eggert <egg...@cs.ucla.edu> wrote:
> Thanks for reporting the problem. Please try the attached patch against > Gawk master. The ChangeLog entry is a bit optimistic, as it is assuming > that the patch works (and if it works, I would like to install the > relevant changes into Gnulib and into glibc, so that at that point the > ChangeLog entry will be correct). Those changes look really excessive to me. I prefer to not have to keep including more and more files from gnulib just to compile regex or dfa. (As a side point, does all the __builtin_expect / __glibc_unlikely stuff *really* make that much difference? It sure clutters up the code unmercifully.) > Alternatively, if you want a smaller patch you can arrange for > __builtin_expect to be a no-op on compilers that do not support it, > as Bruno suggested. That is what I will do, either in my custom.h or directly in regex_internal.h. Thanks, Arnold
