[clang] [HLSL] Enable -fconvergent-functions by default (PR #86571)

2024-03-27 Thread via cfe-commits
https://github.com/aniplcc updated https://github.com/llvm/llvm-project/pull/86571 >From c22f63359b7391e9f69d74e5d4f4bdf6cf585c2c Mon Sep 17 00:00:00 2001 From: aniplcc Date: Tue, 26 Mar 2024 01:35:36 +0530 Subject: [PATCH 1/4] [HLSL] Enable -fconvergent-functions by default ---

[clang] [HLSL] Enable -fconvergent-functions by default (PR #86571)

2024-03-27 Thread via cfe-commits
aniplcc wrote: Ah it's the spir-v check, apologies, forgot to check that https://github.com/llvm/llvm-project/pull/86571 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [HLSL] Enable -fconvergent-functions by default (PR #86571)

2024-03-27 Thread via cfe-commits
aniplcc wrote: check fails with ``` /var/lib/buildkite-agent/builds/linux-56-59b8f5d88-6mhbj-1/llvm-project/github-pull-requests/clang/test/CodeGenHLSL/convergent-functions.hlsl:7:11: error: CHECK: expected string not found in input // CHECK: define void {{.*}}fn{{.*}}() ^ :1:1: note:

[clang] [HLSL] Enable -fconvergent-functions by default (PR #86571)

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

[clang] [HLSL] Enable -fconvergent-functions by default (PR #86571)

2024-03-27 Thread via cfe-commits
aniplcc wrote: Updated https://github.com/llvm/llvm-project/pull/86571 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [HLSL] Enable -fconvergent-functions by default (PR #86571)

2024-03-27 Thread via cfe-commits
https://github.com/aniplcc updated https://github.com/llvm/llvm-project/pull/86571 >From c22f63359b7391e9f69d74e5d4f4bdf6cf585c2c Mon Sep 17 00:00:00 2001 From: aniplcc Date: Tue, 26 Mar 2024 01:35:36 +0530 Subject: [PATCH 1/3] [HLSL] Enable -fconvergent-functions by default ---

[clang] [HLSL] Enable -fconvergent-functions by default (PR #86571)

2024-03-27 Thread Chris B via cfe-commits
llvm-beanz wrote: > I was getting match errors with the RUN script. I went ahead and updated it > with the checks in: `clang/test/CodeGen/convergent-functions.cpp` Hope that's > correct/OK? > > ```hlsl > // RUN: %clang_cc1 -triple dxil-pc-shadermodel6.4-library -emit-llvm >

[clang] [HLSL] Enable -fconvergent-functions by default (PR #86571)

2024-03-27 Thread via cfe-commits
https://github.com/aniplcc updated https://github.com/llvm/llvm-project/pull/86571 >From c22f63359b7391e9f69d74e5d4f4bdf6cf585c2c Mon Sep 17 00:00:00 2001 From: aniplcc Date: Tue, 26 Mar 2024 01:35:36 +0530 Subject: [PATCH 1/2] [HLSL] Enable -fconvergent-functions by default ---

[clang] [HLSL] Enable -fconvergent-functions by default (PR #86571)

2024-03-27 Thread via cfe-commits
aniplcc wrote: I was getting match errors with the RUN script. I went ahead and updated it with the checks in: `clang/test/CodeGen/convergent-functions.cpp` Hope that's correct/OK? ```hlsl // RUN: %clang_cc1 -triple dxil-pc-shadermodel6.4-library -emit-llvm -disable-llvm-passes -o - %s

[clang] [HLSL] Enable -fconvergent-functions by default (PR #86571)

2024-03-26 Thread Chris B via cfe-commits
llvm-beanz wrote: Hi @aniplcc, thank you for the PR. We do require test cases for bug fixes and new features (see the [Developer Policy](https://llvm.org/docs/DeveloperPolicy.html#test-cases). Because this general feature is already tested for OpenCL, we can make a really simple test for

[clang] [HLSL] Enable -fconvergent-functions by default (PR #86571)

2024-03-25 Thread via cfe-commits
aniplcc wrote: Requesting review, @llvm-beanz https://github.com/llvm/llvm-project/pull/86571 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [HLSL] Enable -fconvergent-functions by default (PR #86571)

2024-03-25 Thread via cfe-commits
github-actions[bot] wrote: :white_check_mark: With the latest revision this PR passed the Python code formatter. https://github.com/llvm/llvm-project/pull/86571 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [HLSL] Enable -fconvergent-functions by default (PR #86571)

2024-03-25 Thread via cfe-commits
github-actions[bot] wrote: :white_check_mark: With the latest revision this PR passed the C/C++ code formatter. https://github.com/llvm/llvm-project/pull/86571 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [HLSL] Enable -fconvergent-functions by default (PR #86571)

2024-03-25 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: aniplcc (aniplcc) Changes Fixes #86506 --- Full diff: https://github.com/llvm/llvm-project/pull/86571.diff 1 Files Affected: - (modified) clang/lib/Frontend/CompilerInvocation.cpp (+3-2) ``diff diff --git

[clang] [HLSL] Enable -fconvergent-functions by default (PR #86571)

2024-03-25 Thread via cfe-commits
https://github.com/aniplcc created https://github.com/llvm/llvm-project/pull/86571 Fixes #86506 >From c22f63359b7391e9f69d74e5d4f4bdf6cf585c2c Mon Sep 17 00:00:00 2001 From: aniplcc Date: Tue, 26 Mar 2024 01:35:36 +0530 Subject: [PATCH] [HLSL] Enable -fconvergent-functions by default ---