On Wed, Mar 9, 2016 at 1:37 PM, Francisco Jerez <curroje...@riseup.net> wrote:
> Iago Toral <ito...@igalia.com> writes:
>
>> On Tue, 2016-03-08 at 17:42 -0800, Francisco Jerez wrote:
>>> brw_cfg.h already has include guards, remove the "#pragma once" which
>>> is redundant and non-standard.
>>
>> FWIW, I think using both #pragma once and include guards is a way to
>> keep portability while still getting the performance advantage of
>> #pragma once where it is supported.
>>
> It's highly unlikely to make any significant difference on any
> reasonably modern compiler.  I cannot measure any change in compilation
> time locally from my cleanup.
>
>> Also it seems that we do the same thing in many other files...
>>
> Really?  I'm not aware of any other file where we use both.

There are quite a few in glsl/
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to