[PATCH] D134617: [HLSL] Support register binding attribute on global variable

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 4 11:45:55 PDT 2022


aaron.ballman added inline comments.


================
Comment at: clang/include/clang/Parse/Parser.h:2817-2818
 
+  void MaybeParseHLSLSemantics(Declarator &D,
+                               SourceLocation *EndLoc = nullptr) {
+    if (Tok.is(tok::colon)) {
----------------
aaron.ballman wrote:
> Let's assert we're in HLSL mode here so that we don't accidentally call this from non-HLSL parsing contexts.
Precommit CI finds a way to trigger this assert, so it looks like we caught a case we didn't know about (yay asserts!).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134617/new/

https://reviews.llvm.org/D134617



More information about the cfe-commits mailing list