[all-commits] [llvm/llvm-project] 35fc79: [DirectX] Update CBuffer to refer to a `dx.Layout`...

[Justin Bogner via All-commits]

  Branch: refs/heads/users/bogner/128696+128697+128698
  Home:   https://github.com/llvm/llvm-project
  Commit: 35fc79185613d9b8cf70f8cf0776a8e602c16193
      https://github.com/llvm/llvm-project/commit/35fc79185613d9b8cf70f8cf0776a8e602c16193
  Author: Justin Bogner <mail at justinbogner.com>
  Date:   2025-02-25 (Tue, 25 Feb 2025)

  Changed paths:
    M llvm/include/llvm/Analysis/DXILResource.h
    M llvm/lib/Analysis/DXILResource.cpp
    M llvm/test/Analysis/DXILResource/buffer-frombinding.ll

  Log Message:
  -----------
  [DirectX] Update CBuffer to refer to a `dx.Layout` type

This adds support cbuffers based on llvm/wg-hlsl#171 - the type argument
of the CBuffer TargetExtType is either a `dx.Layout` type which reports
its own size, or it's a normal type and we can simply refer to
DataLayout.


  Commit: a01831471b42e201133bffda63cb8db96997fcba
      https://github.com/llvm/llvm-project/commit/a01831471b42e201133bffda63cb8db96997fcba
  Author: Justin Bogner <mail at justinbogner.com>
  Date:   2025-02-25 (Tue, 25 Feb 2025)

  Changed paths:
    M llvm/lib/Target/DirectX/DXILPrettyPrinter.cpp
    M llvm/test/CodeGen/DirectX/CreateHandleFromBinding.ll

  Log Message:
  -----------
  [DirectX] Fix printing of DXIL cbuffer info

Make sure we're able to print cbuffer comments in a way that's
compatible with DXC.

Fixes #128562


  Commit: 7e00339e259f01f8b18d409760d78eee2460aeee
      https://github.com/llvm/llvm-project/commit/7e00339e259f01f8b18d409760d78eee2460aeee
  Author: Justin Bogner <mail at justinbogner.com>
  Date:   2025-02-25 (Tue, 25 Feb 2025)

  Changed paths:
    M llvm/lib/Analysis/DXILResource.cpp
    A llvm/test/CodeGen/DirectX/Metadata/cbuffer-only.ll

  Log Message:
  -----------
  [DXIL][Analysis] Make sure resource accessors are contiguous

When some resource types were present, but not all of them, we were
ending up in a situation where we would fail to initialize the `FirstX`
variables and get incorrect iterators.

Fixes #128560.


Compare: https://github.com/llvm/llvm-project/compare/35fc79185613%5E...7e00339e259f

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