[all-commits] [llvm/llvm-project] dd0041: pre-req: Add `unbounded` keyword to lexer

Finn Plummer via All-commits all-commits at lists.llvm.org
Fri Apr 25 13:19:19 PDT 2025


  Branch: refs/heads/users/inbelic/pr-137038
  Home:   https://github.com/llvm/llvm-project
  Commit: dd004171f5777dc68b700a5bf0f96c9c703a57c9
      https://github.com/llvm/llvm-project/commit/dd004171f5777dc68b700a5bf0f96c9c703a57c9
  Author: Finn Plummer <finnplummer at microsoft.com>
  Date:   2025-04-25 (Fri, 25 Apr 2025)

  Changed paths:
    M clang/include/clang/Lex/HLSLRootSignatureTokenKinds.def
    M clang/unittests/Lex/LexHLSLRootSignatureTest.cpp

  Log Message:
  -----------
  pre-req: Add `unbounded` keyword to lexer


  Commit: 58d319539faa8522d0d7ed621383d58e034f3e2b
      https://github.com/llvm/llvm-project/commit/58d319539faa8522d0d7ed621383d58e034f3e2b
  Author: Finn Plummer <finnplummer at microsoft.com>
  Date:   2025-04-25 (Fri, 25 Apr 2025)

  Changed paths:
    M clang/include/clang/Parse/ParseHLSLRootSignature.h
    M clang/lib/Parse/ParseHLSLRootSignature.cpp
    M clang/unittests/Parse/ParseHLSLRootSignatureTest.cpp
    M llvm/include/llvm/Frontend/HLSL/HLSLRootSignature.h

  Log Message:
  -----------
  [HLSL][RootSignature] Add parsing of remaining Descriptor Table params

- defines the special values for `DESCRIPTOR_RANGE_OFFSET_APPEND` and
`unbounded` for the `offset` and `numDescriptors` parameters
respectively

- adds these parmaters to the `DescriptorClause` struct and the params
struct

- plugs in parsing of `numDescriptors` and `offset` into
`parseDescriptorTableClauseParams`

- adds corresponding unit tests


  Commit: a9f7a77798a6d14e58bf140af8172a2cc03ed62b
      https://github.com/llvm/llvm-project/commit/a9f7a77798a6d14e58bf140af8172a2cc03ed62b
  Author: Finn Plummer <canadienfinn at gmail.com>
  Date:   2025-04-25 (Fri, 25 Apr 2025)

  Changed paths:
    R clang/include/clang/Parse/ParseHLSLRootSignature.h
    A clang/include/clang/Sema/ParseHLSLRootSignature.h
    M clang/lib/Parse/CMakeLists.txt
    R clang/lib/Parse/ParseHLSLRootSignature.cpp
    M clang/lib/Sema/CMakeLists.txt
    A clang/lib/Sema/ParseHLSLRootSignature.cpp
    M clang/unittests/CMakeLists.txt
    R clang/unittests/Parse/CMakeLists.txt
    R clang/unittests/Parse/ParseHLSLRootSignatureTest.cpp
    M clang/unittests/Sema/CMakeLists.txt
    A clang/unittests/Sema/ParseHLSLRootSignatureTest.cpp

  Log Message:
  -----------
  [NFC][HLSL][RootSignature] Move `HLSLRootSignatureParser` into clangSema

Noting:
- Currently, `HLSLRootSignatureParser` is defined in `clangParse`, as it
would naturally seem an appropriate place to place.

- Surprisingly, `clangParse` has a dependency on `clangSema`. So we can't
introduce a dependency of `clangSema` onto `clangParse`.

- Given the users of `HLSLRootSignatureParser` will be `SemaHLSL` when
parsing from source and `clangFrontend` when we are parsing as a
command line argument.

- Therefore, we are required to move this out of `clangParse` so that
`clangSema` can reference it.

This commit moves `HLSLRootSignatureParser` into `clangSema` so it can
be linked to all its dependencies (`clangFrontend` already depends on
`clangSema`)


Compare: https://github.com/llvm/llvm-project/compare/06e8bf23b389...a9f7a77798a6

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list