[compiler-rt] [rtsan] Add test to ensure coroutines get caught (PR #111049)

via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 3 12:54:24 PDT 2024


davidtrevelyan wrote:

My personal opinion on this one is that whether or not a coroutine allocates memory is irrelevant for realtime sanitizer, so I'm a bit skeptical of the value provided by this test. Memory allocation should be caught by our tests for `malloc` and friends - where that allocation ultimately comes from (at the widest part of the funnel) is imho immaterial. 

In the hypothetical scenario that a valid coroutines implementation is able to begin a coroutine without allocating from the system, would this test start fail erroneously?

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


More information about the llvm-commits mailing list