Corbin Simpson wrote: > Module: Mesa > Branch: master > Commit: 816cb90e7c41ee9cc02a8b12f64b368bd0c2fd03 > URL: > http://cgit.freedesktop.org/mesa/mesa/commit/?id=816cb90e7c41ee9cc02a8b12f64b368bd0c2fd03 > > Author: Alan Coopersmith <[email protected]> > Date: Sun Jan 17 00:55:39 2010 -0800 > > radeon_compiler: include main/compiler.h for compiler portability macros > > Signed-off-by: Alan Coopersmith <[email protected]> > Reviewed-by: Corbin Simpson <[email protected]> > > --- > > .../drivers/dri/r300/compiler/radeon_compiler.h | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/src/mesa/drivers/dri/r300/compiler/radeon_compiler.h > b/src/mesa/drivers/dri/r300/compiler/radeon_compiler.h > index 731adc1..f27f858 100644 > --- a/src/mesa/drivers/dri/r300/compiler/radeon_compiler.h > +++ b/src/mesa/drivers/dri/r300/compiler/radeon_compiler.h > @@ -23,6 +23,8 @@ > #ifndef RADEON_COMPILER_H > #define RADEON_COMPILER_H > > +#include "../../../../main/compiler.h" > +
I'd rather not see relative paths like these in the source code. Don't the exising -I options suffice? Also, isn't the r300 compiler code used by both the old Mesa/DRI driver and the new gallium driver? Perhaps we should include main/compiler.h or pipe/p_compiler.h depending on which driver is being built. -Brian _______________________________________________ mesa-commit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-commit
