Re: [Mesa-dev] [PATCH 3/5] clover: Add support for compiling to native object code v3

2014-10-21 Thread Jan Vesely
On Thu, 2014-10-09 at 11:07 -0400, Tom Stellard wrote: v2: - Split build_module_native() into three separate functions. - Code cleanups. v3: - More cleanups. --- .../state_trackers/clover/llvm/invocation.cpp | 208 -

Re: [Mesa-dev] [PATCH 3/5] clover: Add support for compiling to native object code v3

2014-10-21 Thread Tom Stellard
On Tue, Oct 21, 2014 at 12:16:12PM -0400, Jan Vesely wrote: On Thu, 2014-10-09 at 11:07 -0400, Tom Stellard wrote: v2: - Split build_module_native() into three separate functions. - Code cleanups. v3: - More cleanups. --- .../state_trackers/clover/llvm/invocation.cpp

Re: [Mesa-dev] [PATCH 3/5] clover: Add support for compiling to native object code v3

2014-10-21 Thread Jan Vesely
On Tue, 2014-10-21 at 09:50 -0700, Tom Stellard wrote: [SNIP] + void + init_targets() { + static bool targets_initialized = false; + if (!targets_initialized) { + LLVMInitializeAllTargets(); + LLVMInitializeAllTargetInfos(); +

[Mesa-dev] [PATCH 3/5] clover: Add support for compiling to native object code v3

2014-10-09 Thread Tom Stellard
v2: - Split build_module_native() into three separate functions. - Code cleanups. v3: - More cleanups. --- .../state_trackers/clover/llvm/invocation.cpp | 208 - src/gallium/targets/opencl/Makefile.am | 1 + 2 files changed, 200 insertions(+), 9

Re: [Mesa-dev] [PATCH 3/5] clover: Add support for compiling to native object code v3

2014-10-09 Thread Francisco Jerez
Tom Stellard thomas.stell...@amd.com writes: v2: - Split build_module_native() into three separate functions. - Code cleanups. v3: - More cleanups. Looks good, Reviewed-by: Francisco Jerez curroje...@riseup.net --- .../state_trackers/clover/llvm/invocation.cpp | 208