Skip to content

Conversation

@aadeshps-mcw
Copy link
Contributor

No description provided.

@@ -0,0 +1,48 @@
; RUN: split-file %s %t
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this test both supersets and subset of existing tests. For example we also have here already transcoding/spirvSourceCL2.0.ll and multi_md.ll (and may be more). If we are about to merge this PR - we should:
a. remove transcoding/spirvSourceCL2.0.ll
b. check, what multi_md.ll does and either incorporate in this test or decide to leave multi_md.ll as a stand alone test

Again, there might be more tests

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think i can maybe delete this test as the spirvSourceCLCXX.ll serves the purpose of this test.

@@ -0,0 +1,62 @@
; RUN: llvm-as %s -o %t.bc
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if I understand purpose of this test, is it for zeroinitialized GVs?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants