[libcxx-commits] [libcxx] Fix typeinfo for undefined sanitizer (PR #121228)

Louis Dionne via libcxx-commits libcxx-commits at lists.llvm.org
Fri Jan 10 08:34:02 PST 2025


ldionne wrote:

> It's not, I wrote in description, anyone can just build libc++ from source with -O1 and undefined sanitizer
> But anyway you see what you want to see, so I don't want to continue this discussion.

Ah, no, I simply missed that. I do a lot of reviews every day and the only way that works is for me to read PR descriptions with varied levels of care depending on the PR. In this case, I read too fast and jumped to the links, and I didn't see that you were able to reproduce this issue on non-emscripten. That changes a lot. Let's follow-up in https://github.com/llvm/llvm-project/pull/122389

https://github.com/llvm/llvm-project/pull/121228


More information about the libcxx-commits mailing list