llvmorg-github-actions[bot] wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-hlsl

Author: Helena Kotas (hekota)

<details>
<summary>Changes</summary>

If the repo path contains "main", the test was matching it against the ModuleID 
in the output because it contains a path to the test file, and the follow-up 
checks were failing.

Based on feedback in 
https://github.com/llvm/llvm-project/pull/206596#discussion_r3510725223.

---
Full diff: https://github.com/llvm/llvm-project/pull/207232.diff


1 Files Affected:

- (modified) 
clang/test/CodeGenHLSL/resources/resources-in-structs-method-call.hlsl (+1-1) 


``````````diff
diff --git 
a/clang/test/CodeGenHLSL/resources/resources-in-structs-method-call.hlsl 
b/clang/test/CodeGenHLSL/resources/resources-in-structs-method-call.hlsl
index 29019dd6f99c3..d9b01a0f0141a 100644
--- a/clang/test/CodeGenHLSL/resources/resources-in-structs-method-call.hlsl
+++ b/clang/test/CodeGenHLSL/resources/resources-in-structs-method-call.hlsl
@@ -40,7 +40,7 @@ struct MyConstants {
 
 ConstantBuffer<MyConstants> Constants;
 
-// CHECK-LABEL: main
+// CHECK-LABEL: define internal void @main()()
 [numthreads(4,1,1)]
 void main() {
 // CHECK: [[TMP1:%.*]] = alloca %struct.MyStruct, align 4

``````````

</details>


https://github.com/llvm/llvm-project/pull/207232
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to