[PATCH] ubsan: label messages as coming from ubsan

I've been working on adding UBSan support to CMake (so that errors found
by it can be uploaded to CDash for viewing). One problem is that the
message is very generic. The patch here addes
'UndefinedBehaviorSanitizer' to the message so that they may be detected
more easily.

If something else is wanted (such as the '===' lines from ASan or TSan
to cope with things like printing the backtrace, that would be possible
as well. It would also be nice if there were a blurb on what the error
was rather than just freeform text (similar to heap-use-after-free and
heap-buffer-overflow) to categorize messages.

Thanks,

--Ben

Ben Boeckel (1):
  ubsan: add 'UndefinedBehaviorSanitizer' to messages

lib/ubsan/ubsan_diag.cc | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

This allows detection of them to be much more accurate since 'runtime
error:' and 'note:' are fairly generic.

Signed-off-by: Ben Boeckel <ben.boeckel@kitware.com>

+samsonov

Ben, you’ve forgot to add the patch.
I encourage you to use llvm.org/docs/Phabricator.html for code reviews

–kcc

It's there; both got held up in moderation (I split this single patch
into a cover and the patch since I imagine there's discussion to be had
about messages outside my patch). (Seems I messed up git send-email
anyways since it still put my personal email on this.)

I'll look at Phabricator.

Thanks,

--Ben