[PATCH] D130017: [HLSL] Add RWBuffer default constructor

2022-07-28 Thread Chris Bieneman via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG66eabeb65dc9: [HLSL] Add RWBuffer default constructor (authored by beanz). Changed prior to commit: https://reviews.llvm.org/D130017?vs=445517&id=

[PATCH] D130017: [HLSL] Add RWBuffer default constructor

2022-07-27 Thread Justin Bogner via Phabricator via cfe-commits
bogner accepted this revision. bogner added a comment. This revision is now accepted and ready to land. Is it worth adding a test that calls this constructor / checks that we end up with a call to the builtin? Comment at: clang/lib/Sema/HLSLExternalSemaSource.cpp:155 +

[PATCH] D130017: [HLSL] Add RWBuffer default constructor

2022-07-18 Thread Chris Bieneman via Phabricator via cfe-commits
beanz created this revision. beanz added reviewers: aaron.ballman, Anastasia, kuhar, bogner, python3kgae. Herald added a project: All. beanz requested review of this revision. Herald added a project: clang. This fills out the default constructor for RWBuffer to assign the handle with the result of