[Mesa-dev] [PATCH 09/11] clover: Add constructors to some of the module classes v3

This is for the llvm code that can't use extended initializers. v2: - Use const references for vector arguments - Move constructor defs before data members - Initialize all values in the default constructors v3: - Fix typo --- src/gallium/state_trackers/clover/core/module.hpp | 13

Re: [Mesa-dev] [PATCH 09/11] clover: Add constructors to some of the module classes v2

Tom Stellard writes: > This is for the llvm code that can't use extended initializers. > > v2: > - Use const references for vector arguments > - Move constructor defs before data members > - Initialize all values in the default constructors > --- > src/gallium/state_trackers/clover/core/mo

[Mesa-dev] [PATCH 09/11] clover: Add constructors to some of the module classes v2

This is for the llvm code that can't use extended initializers. v2: - Use const references for vector arguments - Move constructor defs before data members - Initialize all values in the default constructors --- src/gallium/state_trackers/clover/core/module.hpp | 13 + 1 files

Re: [Mesa-dev] [PATCH 09/11] clover: Add constructors to some of the module classes

Hi Tom, thanks for the patches, just a few suggestions inline. Tom Stellard writes: > This is for the llvm code that can't use extended initializers. > --- > src/gallium/state_trackers/clover/core/module.hpp | 13 + > 1 files changed, 13 insertions(+), 0 deletions(-) > > diff --gi

[Mesa-dev] [PATCH 09/11] clover: Add constructors to some of the module classes

This is for the llvm code that can't use extended initializers. --- src/gallium/state_trackers/clover/core/module.hpp | 13 + 1 files changed, 13 insertions(+), 0 deletions(-) diff --git a/src/gallium/state_trackers/clover/core/module.hpp b/src/gallium/state_trackers/clover/core/mo