[clang] [DO not merge] demo test for https://github.com/llvm/llvm-project/pull/140529 (PR #140602)

Qinkun Bao via cfe-commits cfe-commits at lists.llvm.org
Mon May 19 13:53:58 PDT 2025


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


More information about the cfe-commits mailing list