qinkunbao wrote: > This patch should be "[NFC] Precommit test for src:*=sanitize for UBSan" (or > similar), have the test output "passing" for the current build of clang > (i.e., probably "incorrect" output), reviewed and then merged.
Thank you for your feedback. However, I recommend against merging this test. The linked pull request (https://github.com/llvm/llvm-project/pull/140529) introduces a new feature rather than correcting a bug. I believe it's inappropriate to add a test covering a new feature and then submit a separate pull request to "fix" it. https://github.com/llvm/llvm-project/pull/140602 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
