[llvm] [utils][UpdateTestChecks] Warn about possible target triple mismatch (PR #149645)

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 11 06:29:00 PDT 2025


fhahn wrote:

> If I add support for `arm64-apple` prefix then I dont have config to trigger the warning, so the updated test will not test this functionality but an already tested positive flow.



> If I add support for `arm64-apple` prefix then I dont have config to trigger the warning, so the updated test will not test this functionality but an already tested positive flow.

Right, then the test will be updated to show the new behavior, but it still shows the change in behavior for both this patch and the next patch. In general, patches should be accompanied by tests showing the impact, if possible

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


More information about the llvm-commits mailing list