Update on descriptors

This PR improves the descriptors management.

• Fields computation is simplified and corrected in some corner cases
• Descriptor can now be given a name to improve debug/tracing information
• Tiles can also be individually named if CHAMELEON_KERNELS_TRACE cmake option is enabled
• core_ztile interface is updated to accept descriptor and print log calls (can be disable through environment variable CHAMELEON_COREBLAS_SILENT similarly to STARPU_SILENT)
• Recursive descriptors management is introduced as a first step toward recursive algorithm
• A CHAMELEON_Desc_Print function has been added to help debug the descriptor information (tile size, leading dimensions, ...)

Additionally:

• Fix a small issue with flush in starpu
• Silent warnings issued from the hmat merge
• Add the -Werror option to the CI build to avoid warnings
• Add a chameleon_ceil function to simplify computations in algorithms

changed milestone to %Chameleon 1.2.0

added API Feature StarPU labels

approved this merge request

approved this merge request

@all Just to ping everyone and make sure that you all know that the debug level information has been upgraded in the descriptor. It should provide better information in order to help for #72

If you see anything wrong, please answer here, even if the MR is already merged. We'll try to fix it or revert if too problematic.

approved this merge request

added 16 commits

• 9611bdcb - ci: Add -Werror to force people to remove warnings before pushing
• e71ddf8c - openmp/control: minor coding style
• 850b5e43 - starpu/codelet: Remove unused info in lauum
• a442c4c4 - starpu/hmat: Fix compilation warning in the interface
• 6b193b53 - testings: add options to define multi-level nb
• 9b9d774b - common/include: Add a ceil function
• f00c8fa2 - control/desc: Simplify/Fix computation of descriptor fields
• 0dda86f3 - control/desc: Make sure the Destroy function is generic enough to handle both...
• f4b6e9fd - control/desc: add a name into the descriptor structure
• a8087ff2 - control/desc: Add a print function to debug descriptors and an associated...
• 8667b7fe - descriptors: Add recursive descriptors
• 2e09db0d - starpu/descriptor: fix unregister/flush
• e03d0dd1 - coreblas: Handle recursive descriptors
• 8b915c80 - coreblas/trace: add CHAMELEON_KERNELS_TRACE option to name the tiles and print...
• ee32dd91 - coreblas/trace: add an environment variable to enable/disable log info
• 657861cc - starpu: Exploit time name in starpu

Compare with previous version

resolved all threads

enabled an automatic merge when the pipeline for 657861cc succeeds

merged

mentioned in commit eb90f46d

mentioned in merge request !257 (merged)