xiaoxiang781216 commented on pull request #4869: URL: https://github.com/apache/incubator-nuttx/pull/4869#issuecomment-976695593
> What a simple `sim:ostest` can do is to just confirm that 1. The build of the allocator is good This is already covered by all config except(sim:asan) in each PR. > and 2. If we run with `DEBUGASERTION` enabled we can also catch a bunch of issues. Yes, but it's hard to debug the problem without additional information, and asan/ubsan can report more issue(e.g. stack and global overwrite, undefined behavior). > The thing here is that we will have to enable 2 or more options before building. > Another option is built and run sim:ostest and sim:asan, but it look redundant since both run the same ostest program. > In any case, I was just asking if you had the idea of running that as well, it's not a requirement for this PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
