[clang] [clang][Darwin] Remove legacy framework search path logic in the frontend (PR #138234)

2025-05-09 Thread Ian Anderson via cfe-commits
ian-twilightcoder wrote: I tried `GTEST_SHUFFLE=0 GTEST_TOTAL_SHARDS=96 GTEST_SHARD_INDEX=67 ./AllClangUnitTests` and that also passes. There's a leak from the new `SearchPathTest`, maybe that's making the address sanitizer mad? https://github.com/llvm/llvm-project/pull/138234

[clang] [clang][Darwin] Remove legacy framework search path logic in the frontend (PR #138234)

2025-05-09 Thread Ian Anderson via cfe-commits
ian-twilightcoder wrote: That passes for me. I can't even figure out which unit test is failing, they all say `OK` and not `FAILED`. The one error in there looks like it's expected since its test still returns `OK`. Are we totally sure this isn't an infrastructure issue? https://github.com/ll

[clang] [clang][Darwin] Minor args cleanup (PR #139142)

2025-05-08 Thread Ian Anderson via cfe-commits
https://github.com/ian-twilightcoder created https://github.com/llvm/llvm-project/pull/139142 I just realized that ArgList.hasArg takes multiple arguments. Consolidate the two calls into one. Rate limit ยท GitHub body { background-color: #f6f8fa;

<    1   2   3   4