<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">
Commit comment should not include “[rasterizer codegen]”, as it doesn’t modify that code.
<div class=""><br class="">
</div>
<div class="">With that fixed, Reviewed-by: Tim Rowley <<a href="mailto:timothy.o.rowley@intel.com" class="">timothy.o.rowley@intel.com</a>></div>
<div class=""><br class="">
<div>
<blockquote type="cite" class="">
<div class="">On Mar 28, 2017, at 4:44 PM, George Kyriazis <<a href="mailto:george.kyriazis@intel.com" class="">george.kyriazis@intel.com</a>> wrote:</div>
<br class="Apple-interchange-newline">
<div class="">
<div class="">Fix codegen build break that was introduced earlier<br class="">
<br class="">
v2: update rules for gen_knobs.cpp and gen_knobs.h<br class="">
<br class="">
v3: Introduce bldroot and revert generator file changes, making patch simpler.<br class="">
---<br class="">
src/gallium/drivers/swr/SConscript | 38 +++++++++++++++++++++++++++++++-------<br class="">
1 file changed, 31 insertions(+), 7 deletions(-)<br class="">
<br class="">
diff --git a/src/gallium/drivers/swr/SConscript b/src/gallium/drivers/swr/SConscript<br class="">
index ad16162..18d6c9b 100644<br class="">
--- a/src/gallium/drivers/swr/SConscript<br class="">
+++ b/src/gallium/drivers/swr/SConscript<br class="">
@@ -47,20 +47,25 @@ if not env['msvc'] :<br class="">
    ])<br class="">
<br class="">
swrroot = '#src/gallium/drivers/swr/'<br class="">
+bldroot = Dir('.').abspath<br class="">
<br class="">
env.CodeGenerate(<br class="">
    target = 'rasterizer/codegen/gen_knobs.cpp',<br class="">
    script = swrroot + 'rasterizer/codegen/gen_knobs.py',<br class="">
-    source = 'rasterizer/codegen/templates/gen_knobs.cpp',<br class="">
-    command = python_cmd + ' $SCRIPT --input $SOURCE --output $TARGET --gen_cpp'<br class="">
+    source = '',<br class="">
+    command = python_cmd + ' $SCRIPT --output $TARGET --gen_cpp'<br class="">
)<br class="">
+Depends('rasterizer/codegen/gen_knobs.cpp',<br class="">
+        swrroot + 'rasterizer/codegen/templates/gen_knobs.cpp')<br class="">
<br class="">
env.CodeGenerate(<br class="">
    target = 'rasterizer/codegen/gen_knobs.h',<br class="">
    script = swrroot + 'rasterizer/codegen/gen_knobs.py',<br class="">
-    source = 'rasterizer/codegen/templates/gen_knobs.cpp',<br class="">
-    command = python_cmd + ' $SCRIPT --input $SOURCE --output $TARGET --gen_h'<br class="">
+    source = '',<br class="">
+    command = python_cmd + ' $SCRIPT --output $TARGET --gen_h'<br class="">
)<br class="">
+Depends('rasterizer/codegen/gen_knobs.cpp',<br class="">
+        swrroot + 'rasterizer/codegen/templates/gen_knobs.cpp')<br class="">
<br class="">
env.CodeGenerate(<br class="">
    target = 'rasterizer/jitter/gen_state_llvm.h',<br class="">
@@ -68,20 +73,26 @@ env.CodeGenerate(<br class="">
    source = 'rasterizer/core/state.h',<br class="">
    command = python_cmd + ' $SCRIPT --input $SOURCE --output $TARGET'<br class="">
)<br class="">
+Depends('rasterizer/jitter/gen_state_llvm.h',<br class="">
+        swrroot + 'rasterizer/codegen/templates/gen_llvm.hpp')<br class="">
<br class="">
env.CodeGenerate(<br class="">
    target = 'rasterizer/jitter/gen_builder.hpp',<br class="">
    script = swrroot + 'rasterizer/codegen/gen_llvm_ir_macros.py',<br class="">
    source = os.path.join(llvm_includedir, 'llvm/IR/IRBuilder.h'),<br class="">
-    command = python_cmd + ' $SCRIPT --input $SOURCE --output rasterizer/jitter --gen_h'<br class="">
+    command = python_cmd + ' $SCRIPT --input $SOURCE --output ' + bldroot + '/rasterizer/jitter --gen_h'<br class="">
)<br class="">
+Depends('rasterizer/jitter/gen_builder.hpp',<br class="">
+        swrroot + 'rasterizer/codegen/templates/gen_builder.hpp')<br class="">
<br class="">
env.CodeGenerate(<br class="">
    target = 'rasterizer/jitter/gen_builder_x86.hpp',<br class="">
    script = swrroot + 'rasterizer/codegen/gen_llvm_ir_macros.py',<br class="">
    source = '',<br class="">
-    command = python_cmd + ' $SCRIPT --output rasterizer/jitter --gen_x86_h'<br class="">
+    command = python_cmd + ' $SCRIPT --output ' + bldroot + '/rasterizer/jitter --gen_x86_h'<br class="">
)<br class="">
+Depends('rasterizer/jitter/gen_builder.hpp',<br class="">
+        swrroot + 'rasterizer/codegen/templates/gen_builder.hpp')<br class="">
<br class="">
env.CodeGenerate(<br class="">
    target = './gen_swr_context_llvm.h',<br class="">
@@ -89,6 +100,8 @@ env.CodeGenerate(<br class="">
    source = 'swr_context.h',<br class="">
    command = python_cmd + ' $SCRIPT --input $SOURCE --output $TARGET'<br class="">
)<br class="">
+Depends('rasterizer/jitter/gen_state_llvm.h',<br class="">
+        swrroot + 'rasterizer/codegen/templates/gen_llvm.hpp')<br class="">
<br class="">
env.CodeGenerate(<br class="">
    target = 'rasterizer/archrast/gen_ar_event.hpp',<br class="">
@@ -96,6 +109,8 @@ env.CodeGenerate(<br class="">
    source = 'rasterizer/archrast/events.proto',<br class="">
    command = python_cmd + ' $SCRIPT --proto $SOURCE --output $TARGET --gen_event_h'<br class="">
)<br class="">
+Depends('rasterizer/jitter/gen_state_llvm.h',<br class="">
+        swrroot + 'rasterizer/codegen/templates/gen_ar_event.hpp')<br class="">
<br class="">
env.CodeGenerate(<br class="">
    target = 'rasterizer/archrast/gen_ar_event.cpp',<br class="">
@@ -103,6 +118,8 @@ env.CodeGenerate(<br class="">
    source = 'rasterizer/archrast/events.proto',<br class="">
    command = python_cmd + ' $SCRIPT --proto $SOURCE --output $TARGET --gen_event_cpp'<br class="">
)<br class="">
+Depends('rasterizer/jitter/gen_state_llvm.h',<br class="">
+        swrroot + 'rasterizer/codegen/templates/gen_ar_event.cpp')<br class="">
<br class="">
env.CodeGenerate(<br class="">
    target = 'rasterizer/archrast/gen_ar_eventhandler.hpp',<br class="">
@@ -110,6 +127,8 @@ env.CodeGenerate(<br class="">
    source = 'rasterizer/archrast/events.proto',<br class="">
    command = python_cmd + ' $SCRIPT --proto $SOURCE --output $TARGET --gen_eventhandler_h'<br class="">
)<br class="">
+Depends('rasterizer/jitter/gen_state_llvm.h',<br class="">
+        swrroot + 'rasterizer/codegen/templates/gen_ar_eventhandler.hpp')<br class="">
<br class="">
env.CodeGenerate(<br class="">
    target = 'rasterizer/archrast/gen_ar_eventhandlerfile.hpp',<br class="">
@@ -117,6 +136,8 @@ env.CodeGenerate(<br class="">
    source = 'rasterizer/archrast/events.proto',<br class="">
    command = python_cmd + ' $SCRIPT --proto $SOURCE --output $TARGET --gen_eventhandlerfile_h'<br class="">
)<br class="">
+Depends('rasterizer/jitter/gen_state_llvm.h',<br class="">
+        swrroot + 'rasterizer/codegen/templates/gen_ar_eventhandlerfile.hpp')<br class="">
<br class="">
# 5 SWR_MULTISAMPLE_TYPE_COUNT<br class="">
# 2 SWR_MSAA_SAMPLE_PATTERN_COUNT<br class="">
@@ -127,8 +148,11 @@ env.CodeGenerate(<br class="">
env.CodeGenerate(<br class="">
    target = 'rasterizer/core/gen_BackendPixelRate0.cpp',<br class="">
    script = swrroot + 'rasterizer/codegen/gen_backends.py',<br class="">
-    command = python_cmd + ' $SCRIPT --output rasterizer/core --dim 5 2 3 2 2 2 --split 0 --cpp'<br class="">
+    source = '',<br class="">
+    command = python_cmd + ' $SCRIPT --outdir ' + bldroot + '/rasterizer/core --dim 5 2 3 2 2 2 --split 0 --cpp'<br class="">
)<br class="">
+Depends('rasterizer/jitter/gen_state_llvm.h',<br class="">
+        swrroot + 'rasterizer/codegen/templates/gen_backend.cpp')<br class="">
<br class="">
# Auto-generated .cpp files (that need to generate object files)<br class="">
built_sources = [<br class="">
-- <br class="">
2.10.1 (Apple Git-78)<br class="">
<br class="">
_______________________________________________<br class="">
mesa-dev mailing list<br class="">
<a href="mailto:mesa-dev@lists.freedesktop.org" class="">mesa-dev@lists.freedesktop.org</a><br class="">
https://lists.freedesktop.org/mailman/listinfo/mesa-dev<br class="">
</div>
</div>
</blockquote>
</div>
<br class="">
</div>
</body>
</html>