[all-commits] [llvm/llvm-project] 6992eb: Reapply "[DXIL][Analysis] Make alignment on Struct...

[Justin Bogner via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6992ebcb4b69fc7f116cd475e4731d3877118c1d
      https://github.com/llvm/llvm-project/commit/6992ebcb4b69fc7f116cd475e4731d3877118c1d
  Author: Justin Bogner <mail at justinbogner.com>
  Date:   2024-07-30 (Tue, 30 Jul 2024)

  Changed paths:
    M llvm/include/llvm/Analysis/DXILResource.h
    M llvm/lib/Analysis/DXILResource.cpp
    M llvm/unittests/Analysis/DXILResourceTest.cpp

  Log Message:
  -----------
  Reapply "[DXIL][Analysis] Make alignment on StructuredBuffer optional" (#101113)

Unfortunately storing a `MaybeAlign` in ResourceInfo deletes our move
constructor in compilers that haven't implemented [P0602R4], like GCC 7.
Since we only ever use the alignment in ways where alignment 1 and unset
are ambiguous anyway, we'll just store the integer AlignLog2 value that
we'll eventually use directly.

[P0602R4]:
https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2018/p0602r4.html

This reverts commit c22171f12fa9f260e2525cf61b93c136889e17f2, reapplying
a94edb6b8e321a46fe429934236aaa4e2e9fb97f.



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