On 28/05/15 12:28 AM, Gupta, Maneesh wrote: > Attached is a patch that enables configure to autodetect the presence of > OpenCL headers and ICD loader. If the necessary headers are found, the OpenCL > infrastructure compilation is enabled as well. Note that this does not modify > the runtime behavior in any way. Execution of the OpenCL code path still > requires the user to explicitly enable it. > > Please review the patch and share your comments on the same. > > - Maneesh
I personally don't agree OpenCL should be autodetected. Currently, only the so called "system" libs are autodetected (zlib, iconv, pthreads, the like). Open*L, decoding/encoding libraries, and other miscellaneous libraries are enabled only if explicitly requested. If we enable one by default, then there's no argument against enabling every other as well ("Why this one but not that other one?"), and suddenly default builds become bloated and automated scripts get potentially broken until adapted. Lets keep default builds as lightweight as possible, and allow people to explicitly enable the functionality they require. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel