On Tue, Jul 26, 2016 at 04:21:26AM +0200, Dieter Nützel wrote: > Signed-off-by: Dieter Nützel <die...@nuetzel-hh.de>
The change itself looks good, but could you replace "c++ 4.8" with "GCC 4.8" in the commit title, and add a commit message with the error? For reference: error: invalid initialization of non-const reference of type 'clover::llvm::compat::raw_ostream_to_emit_file {aka llvm::raw_svector_ostream&}' from an rvalue of type '<brace-enclosed initializer list>' > --- > src/gallium/state_trackers/clover/llvm/codegen/native.cpp | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/gallium/state_trackers/clover/llvm/codegen/native.cpp > b/src/gallium/state_trackers/clover/llvm/codegen/native.cpp > index b96236b..f5e887e 100644 > --- a/src/gallium/state_trackers/clover/llvm/codegen/native.cpp > +++ b/src/gallium/state_trackers/clover/llvm/codegen/native.cpp > @@ -126,7 +126,7 @@ namespace { > { > compat::pass_manager pm; > ::llvm::raw_svector_ostream os { data }; > - compat::raw_ostream_to_emit_file fos { os }; > + compat::raw_ostream_to_emit_file fos ( os ); > > mod.setDataLayout(compat::get_data_layout(*tm)); > tm->Options.MCOptions.AsmVerbose = > -- > 2.1.4 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev