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(); > > > +

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/invo

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 > - > src/gallium/targets/opencl/M

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

2014-10-09 Thread Francisco Jerez
Tom Stellard writes: > v2: > - Split build_module_native() into three separate functions. > - Code cleanups. > > v3: > - More cleanups. Looks good, Reviewed-by: Francisco Jerez > --- > .../state_trackers/clover/llvm/invocation.cpp | 208 > - > src/gallium/targe

[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 deleti