<div dir="ltr">Thanks for the review!<div><br><div>These patches, along with 5 more I've put together, </div><div>don't have any significant impact on compile time.</div><div>But that's only 16 files, so it really shouldn't be expected I guess.</div>
<div>A similar effort on firefox gave a 12.5% compile-time improvement though,</div><div>so eventually we should be seeing some effects. [1]</div><div><br></div><div>Where I do see an impact though is recompiling after changes.</div>
<div>Less object-files needs to be recompiled, so it's often a lot faster.</div><div>I don't have any specific numbers here though.</div><div><br></div><div>I see that I've managed to get a "From: ...." line in my mails, </div>
<div>and sender is not showing up correctly in the mailing-list.</div><div>Anyone git-savvy who've had similar problems?</div><div>If not I'll delve into the dark caves of man-pages and read up.</div><div><br></div>
<div>[1] <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=903816">https://bugzilla.mozilla.org/show_bug.cgi?id=903816</a></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-06-10 19:58 GMT+02:00 Tom Stellard <span dir="ltr"><<a href="mailto:tom@stellard.net" target="_blank">tom@stellard.net</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Jun 10, 2014 at 12:57:31AM +0200, <a href="mailto:thomashelland90@gmail.com">thomashelland90@gmail.com</a> wrote:<br>

> From: Thomas Helland <<a href="mailto:thomashelland90@gmail.com">thomashelland90@gmail.com</a>><br>
><br>
<br>
For the series:<br>
<br>
Reviewed-by: Tom Stellard <<a href="mailto:thomas.stellard@amd.com">thomas.stellard@amd.com</a>><br>
<br>
Do these patches make any measurable improvement in compile time?<br>
<br>
-Tom<br>
> This series removes some unused includes in the glsl-directory.<br>
> These were found with Google's include-what-you-use plugin for clang.<br>
> Patches have been compile-tested and a quick glxgears-run has been done.<br>
> I have not done a full piglit-run, let me know if that's wanted.<br>
><br>
> Oh, and excuse me if I've messed up something.<br>
> First run with git and mailing-lists, so lots to learn.<br>
><br>
> Thomas Helland (11):<br>
>   glsl: Remove unused includes in link_uniform_init.<br>
>   glsl: Remove unused includes in link_uniform_block_active_visitor.h<br>
>   glsl: Remove unused include in ast_to_hir.cpp<br>
>   glsl: Remove unused include from builtin_variables.cpp<br>
>   glsl: Remove unused include from glsl_types.cpp<br>
>   glsl: Remove unused include from glsl_symbol_table.h<br>
>   glsl: Remove unused include from hir_field_selection.cpp<br>
>   glsl: Remove unused include from ir_basic_block.cpp<br>
>   glsl: Remove unused include from ir_constant_expression.cpp<br>
>   glsl: Remove unused include in ir.cpp<br>
>   glsl: Remove unused include in expr.flatt.<br>
><br>
>  src/glsl/ast_to_hir.cpp                      | 1 -<br>
>  src/glsl/builtin_variables.cpp               | 1 -<br>
>  src/glsl/glsl_symbol_table.h                 | 2 +-<br>
>  src/glsl/glsl_types.cpp                      | 4 +---<br>
>  src/glsl/hir_field_selection.cpp             | 1 -<br>
>  src/glsl/ir.cpp                              | 1 -<br>
>  src/glsl/ir_basic_block.cpp                  | 2 --<br>
>  src/glsl/ir_constant_expression.cpp          | 1 -<br>
>  src/glsl/ir_expression_flattening.cpp        | 2 --<br>
>  src/glsl/link_uniform_block_active_visitor.h | 2 --<br>
>  src/glsl/link_uniform_initializers.cpp       | 2 --<br>
>  11 files changed, 2 insertions(+), 17 deletions(-)<br>
<span class="HOEnZb"><font color="#888888">><br>
> --<br>
> 2.0.0<br>
><br>
> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div>