Skip to content
The Jenkins Controller is preparing for shutdown. No new builds can be started.
Success

Changes

Summary

  1. Fix C build warnings. (details)
  2. Load jocl.properties if present. (details)
  3. Pass function pointers into clEnqueueMapImage. (details)
  4. Remove unnamed struct emission warnings. (details)
  5. Remove alignment warning in gluegen preprocessing. (details)
  6. Remove warnings, turn on linting, fix Eclipse. (details)
  7. Remove Java lint warnings. (details)
  8. Fix crashes due to AMD driver bugs. (details)
  9. Fix transposed arguments in putCopyBufferRect (details)
Commit 9d6418135370a8ce1d5c63196011b1354a8e8022 by Wade Walker
Fix C build warnings.

Fixes -fno-rtti and implicit cast warnings. After this commit,
all C warnings are gone except for one due to gluegen emitting
char pointers instead of unsigned char pointers.
The file was modifiedmake/build.xml (diff)
The file was modifiedmake/config/clImplCustomCode.c (diff)
Commit eb06798a7af0da8febbf31abd34e62e7100633aa by Wade Walker
Load jocl.properties if present.

This lets us build with debug information turned on.
The file was modifiedmake/build.xml (diff)
Commit 806f2902482af7c77c6a25ac3f9e4d4f73d56a54 by Wade Walker
Pass function pointers into clEnqueueMapImage.

These pointers were showing up as uninitialized variables; on inspection
they just weren't being passed in from the Java side or assigned on
the C side. There are currently no tests of this function, which is how
we didn't notice this omission.
The file was modifiedmake/config/clImplCustomCode.c (diff)
The file was modifiedsrc/com/jogamp/opencl/llb/impl/CLImpl.java (diff)
Commit ddfd2364cd95603f059169b44666b20ba54069f2 by Wade Walker
Remove unnamed struct emission warnings.

Uses the new augmentation of gluegen's Ignore keyword.
The file was modifiedmake/config/cl-common.cfg (diff)
Commit 6b865b8650b62f448760cfbe50cc563f7ab85da9 by Wade Walker
Remove alignment warning in gluegen preprocessing.

Turns off the warning about CL_ALIGNED not being
defined for the current compiler, but only inside
the gluegen preprocessor. If CL_ALIGNED isn't
defined for the real C compiler on the platform,
the warning will still show up during C compilation.
The file was modifiedmake/stub_includes/CL_orig/cl_platform.h (diff)
Commit 8c406de8eb50cf785b407e2facb3502e364a66ce by Wade Walker
Remove warnings, turn on linting, fix Eclipse.

Removed warnings due to memoryMaximumSize being ignored when javac isn't
forked. Turned on all linting. Changed the arguments of
uncomment-function-params so it will run from within Eclipse properly
(using basedir instead of user.dir, since only the former will be
properly set both inside Eclipse and on the command line).
The file was modifiedmake/build.xml (diff)
The file was modifiedmake/build-test.xml (diff)
Commit 7ab26044167c84fc6386cc179e8a8736d8978c91 by Wade Walker
Remove Java lint warnings.

Remove all Java lint warnings, by fixing the code if possible, and if
not possible then by inserting @SuppressWarnings. Some of these
@SuppressWarnings can be replaced later with @SafeVarargs if we
eventually drop support for Java 6.
The file was modifiedsrc/com/jogamp/opencl/util/concurrent/CLCommandQueuePool.java (diff)
The file was modifiedsrc/com/jogamp/opencl/util/CLMultiContext.java (diff)
The file was modifiedtest/com/jogamp/opencl/CLBufferTest.java (diff)
The file was modifiedsrc/com/jogamp/opencl/AutoCloseable.jtemplate (diff)
The file was modifiedsrc/com/jogamp/opencl/CLPlatform.java (diff)
The file was modifiedtest/com/jogamp/opencl/HighLevelBindingTest.java (diff)
The file was modifiedtest/com/jogamp/opencl/gl/CLGLTest.java (diff)
The file was modifiedsrc/com/jogamp/opencl/util/JOCLVersion.java (diff)
The file was modifiedsrc/com/jogamp/opencl/CLEventList.java (diff)
The file was modifiedtest/com/jogamp/opencl/CLCommandQueueTest.java (diff)
Commit 7a7f87a1fc7419f758ba9b134764ae544fd6d566 by Wade Walker
Fix crashes due to AMD driver bugs.

programBinariesTest() failure was due to AMD drivers crashing
in clCreateKernelsInProgram() when the program is not built yet,
instead of returning error code CL_INVALID_PROGRAM_EXECUTABLE as they
should.

lowLevelVectorAddTest() failure was apparently due to the AMD drivers
writing past the end of a direct byte buffer in such a way that it made
System.gc() crash when called during teardown (this crash didn't even
dump stack). Making the buffer larger solved the problem.
The file was modifiedsrc/com/jogamp/opencl/CLPlatform.java (diff)
The file was modifiedtest/com/jogamp/opencl/CLProgramTest.java (diff)
The file was modifiedtest/com/jogamp/opencl/LowLevelBindingTest.java (diff)
Commit 65144f3de56b4fc47262399d0b3bcb0fa5f3feee by Wade Walker
Fix transposed arguments in putCopyBufferRect

Fixes transposed arguments in CLCommandQueue:putCopyBufferRect.
The file was modifiedsrc/com/jogamp/opencl/CLCommandQueue.java (diff)