svn commit: r265367 - head/lib/libc/regex

David Chisnall theraven at FreeBSD.org
Mon May 5 21:24:43 UTC 2014


On 5 May 2014, at 20:49, Pedro Giffuni <pfg at FreeBSD.org> wrote:

> Yes, but I reverted it because there are other ways to check for overflows without the performance hit.

Do we have a good reusable routine for doing this somewhere?  Clang and gcc both have some idiom recognisers that try to spot when people are attempting to do this.  Clang also has a builtin, which would be good to use when available.  Overflow checking is very cheap on modern CPUs (add, branch on carry), so it would be nice if we could start looking for this malloc() and realloc() pattern and replacing the multiply with something that checks for the error.

David



More information about the svn-src-all mailing list