Skip to content

Commit

Permalink
spirv: fix build for C++20 (#6105)
Browse files Browse the repository at this point in the history
This codebase is using C++17, but to remain compatible with C++20, I
need to remove this.
Starting C++20, an aggregate is required to have no user-declared
constructors, vs before, no user-provided constructors.

So even if if the only the default constructor is provided, this is
enough to prevent us from using this class as an aggregate in c++20.

Signed-off-by: Nathan Gauër <brioche@google.com>
  • Loading branch information
Keenuts authored Dec 18, 2023
1 parent 3ef6ca2 commit a743e97
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions tools/clang/lib/SPIRV/DeclResultIdMapper.h
Original file line number Diff line number Diff line change
Expand Up @@ -384,8 +384,6 @@ class DeclResultIdMapper {
/// The struct containing the data needed to create the input and output
/// variables for the decl.
struct StageVarDataBundle {
public:
StageVarDataBundle() = default;
// The declaration of the variable for which we need to create the stage
// variables.
const NamedDecl *decl;
Expand Down

0 comments on commit a743e97

Please sign in to comment.