Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/test/CodeGen/AMDGPU/function-returns.ll | ||
---|---|---|
1 ↗ | (On Diff #530872) | Switching to generated checks should be done in a precommit |
llvm/test/CodeGen/AMDGPU/function-returns.ll | ||
---|---|---|
1 ↗ | (On Diff #530872) | I don't see how that can be useful in this case. There are no functional changes and the new checks are all as visible here as they would be in a separate patch. |
llvm/test/CodeGen/AMDGPU/function-returns.ll | ||
---|---|---|
1 ↗ | (On Diff #530872) | I think adding a GFX11 runline to existing tests (manual or autogenerated) is fine as long as the test is relevant to GFX11. The questionable part for me comes in when we are converting manually written tests to autogenerated. These need some level of judgement to determine if the clarity of the test is lost by making it auto generated. I think it would ease review if you resorted tests in the unlanded patches in this series into a format where a patch contains only one of 1) adding GFX11 runlines or 2) converting a test to autogenerated and adding GFX11 runlines. Patches with 1) only should be easy to approve. Then we can take a closer look at if those cases in 2) should really be manually updated. |