[llvm-commits] [PATCH] Add -fcatch-undefined-behavior runtime library

[Richard Smith]

On Thu, Oct 4, 2012 at 3:58 AM, Kostya Serebryany <kcc at google.com> wrote:

> >> +void __ubsan::__ubsan_handle_divrem_overflow
> For consistency with asan/tsan you may want to move public functions out
> of __ubsan namespace (just have them __ubsan_foo).
> The "__" guaranties that no legal program will have the same symbol, while
> the simple extern "C" name may slightly simplify debugging (e.g. setting a
> break point, etc)
>

Since these symbols are already extern "C", their mangled names are just
__ubsan_handle_divrem_overflow etc, so there's no difference from a
debugging perspective. Is there a benefit to moving them out of the
namespace?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121004/b09b12be/attachment.html>