[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