[clang] [HLSL] standardize builtin unit tests (PR #83340)

2024-02-29 Thread Xiang Li via cfe-commits
https://github.com/python3kgae closed https://github.com/llvm/llvm-project/pull/83340 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [HLSL] standardize builtin unit tests (PR #83340)

2024-02-29 Thread Chris B via cfe-commits
https://github.com/llvm-beanz approved this pull request. https://github.com/llvm/llvm-project/pull/83340 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [HLSL] standardize builtin unit tests (PR #83340)

2024-02-29 Thread Damyan Pepper via cfe-commits
https://github.com/damyanp approved this pull request. https://github.com/llvm/llvm-project/pull/83340 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [HLSL] standardize builtin unit tests (PR #83340)

2024-02-28 Thread Justin Bogner via cfe-commits
https://github.com/bogner approved this pull request. https://github.com/llvm/llvm-project/pull/83340 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [HLSL] standardize builtin unit tests (PR #83340)

2024-02-28 Thread Justin Bogner via cfe-commits
bogner wrote: If we want to talk about guidelines for clang-formatting tests in general we should probably bring it up with the wider LLVM community on discourse. The de facto standard today is that tests should be well formatted like anything else unless there's a good reason not to, but

[clang] [HLSL] standardize builtin unit tests (PR #83340)

2024-02-28 Thread Farzon Lotfi via cfe-commits
farzonl wrote: > Do we have the suggestions that this is responding to written down somewhere? > I think it would be useful to have those guidelines for anyone who might want > to contribute HLSL tests. At any rate, I'd like to know the ones that this is > in response to. After consulting

[clang] [HLSL] standardize builtin unit tests (PR #83340)

2024-02-28 Thread Farzon Lotfi via cfe-commits
farzonl wrote: > > Do we have the suggestions that this is responding to written down > > somewhere? I think it would be useful to have those guidelines for anyone > > who might want to contribute HLSL tests. At any rate, I'd like to know the > > ones that this is in response to. > >

[clang] [HLSL] standardize builtin unit tests (PR #83340)

2024-02-28 Thread Farzon Lotfi via cfe-commits
farzonl wrote: > Do we have the suggestions that this is responding to written down somewhere? > I think it would be useful to have those guidelines for anyone who might want > to contribute HLSL tests. At any rate, I'd like to know the ones that this is > in response to. @pow2clk below are

[clang] [HLSL] standardize builtin unit tests (PR #83340)

2024-02-28 Thread Damyan Pepper via cfe-commits
damyanp wrote: > Do we have the suggestions that this is responding to written down somewhere? > I think it would be useful to have those guidelines for anyone who might want > to contribute HLSL tests. At any rate, I'd like to know the ones that this is > in response to. Following on from

[clang] [HLSL] standardize builtin unit tests (PR #83340)

2024-02-28 Thread Greg Roth via cfe-commits
pow2clk wrote: Do we have the suggestions that this is responding to written down somewhere? I think it would be useful to have those guidelines for anyone who might want to contribute HLSL tests. At any rate, I'd like to know the ones that this is in response to.

[clang] [HLSL] standardize builtin unit tests (PR #83340)

2024-02-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-hlsl @llvm/pr-subscribers-clang Author: Farzon Lotfi (farzonl) Changes This PR brings best practices mentioned to me on other prs and adds them to the existing builtin tests. Note to reviewers: I put this up in two commits because the clang-format

[clang] [HLSL] standardize builtin unit tests (PR #83340)

2024-02-28 Thread Farzon Lotfi via cfe-commits
https://github.com/farzonl created https://github.com/llvm/llvm-project/pull/83340 This PR brings best practices mentioned to me on other prs and adds them to the existing builtin tests. Note to reviewers: I put this up in two commits because the clang-format changes is making it hard to