Getting rid of the builtin_compiler from the Android build is a good
idea... but I'll let Chad or someone who knows /anything/ about the
Android build system actually review the patch.

Acked-by: Ian Romanick <ian.d.roman...@intel.com>

I don't think I want to remove any of the other bits of the builtin
compiler or standalone compiler.  I've been working on resurrecting the
standalone compiler for use as a developer tool.

http://cgit.freedesktop.org/~idr/mesa/log/?h=standalone-compiler

On 09/13/2013 03:58 AM, Adrian Negreanu wrote:
> From: Adrian Negreanu <adrian.m.negre...@intel.com>
> 
> the first part was done in:
> 
>    commit c845140a20efa6a30a5465301d1f9b4acea79155
>    Author: Kenneth Graunke <kenn...@whitecape.org>
>    Date:   Tue Sep 3 21:22:17 2013 -0700
> 
> Signed-off-by: Adrian Negreanu <adrian.m.negre...@intel.com>
> ---
>  src/glsl/Android.gen.mk | 19 -------------------
>  1 file changed, 19 deletions(-)
> 
> diff --git a/src/glsl/Android.gen.mk b/src/glsl/Android.gen.mk
> index 2ed9fa6..7ec56d4 100644
> --- a/src/glsl/Android.gen.mk
> +++ b/src/glsl/Android.gen.mk
> @@ -35,10 +35,6 @@ sources := \
>       glcpp/glcpp-lex.c \
>       glcpp/glcpp-parse.c
>  
> -ifneq ($(LOCAL_IS_HOST_MODULE),true)
> -sources += builtin_function.cpp
> -endif
> -
>  LOCAL_SRC_FILES := $(filter-out $(sources), $(LOCAL_SRC_FILES))
>  
>  LOCAL_C_INCLUDES += $(intermediates) $(intermediates)/glcpp 
> $(MESA_TOP)/src/glsl/glcpp
> @@ -81,18 +77,3 @@ $(intermediates)/glcpp/glcpp-lex.c: 
> $(LOCAL_PATH)/glcpp/glcpp-lex.l
>  
>  $(intermediates)/glcpp/glcpp-parse.c: $(LOCAL_PATH)/glcpp/glcpp-parse.y
>       $(call glsl_local-y-to-c-and-h)
> -
> -BUILTIN_COMPILER := 
> $(BUILD_OUT_EXECUTABLES)/mesa_builtin_compiler$(BUILD_EXECUTABLE_SUFFIX)
> -
> -builtin_function_deps := \
> -     $(LOCAL_PATH)/builtins/tools/generate_builtins.py \
> -     $(LOCAL_PATH)/builtins/tools/texture_builtins.py \
> -     $(BUILTIN_COMPILER) \
> -     $(wildcard $(LOCAL_PATH)/builtins/profiles/*) \
> -             $(wildcard $(LOCAL_PATH)/builtins/ir/*)
> -
> -$(intermediates)/builtin_function.cpp: PRIVATE_SCRIPT := $(MESA_PYTHON2) 
> $(LOCAL_PATH)/builtins/tools/generate_builtins.py
> -$(intermediates)/builtin_function.cpp: $(builtin_function_deps)
> -     @mkdir -p $(dir $@)
> -     @echo "Gen GLSL: $(PRIVATE_MODULE) <= $(notdir $@)"
> -     $(hide) $(PRIVATE_SCRIPT) $(BUILTIN_COMPILER) > $@ || rm -f $@
> 

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to