CHOLMOD GPU: Fix openmp

[whuaegeanse]

On windows, MSVC throws openmp related compilation errors

[mmuetzel]

I can only guess which error you are seeing. But would it be better to make num_chunks a size_t instead of making i an int?
That might help with integer overflow for large numbers.

[whuaegeanse]

I can only guess which error you are seeing. But would it be better to make num_chunks a size_t instead of making i an int? That might help with integer overflow for large numbers.

@mmuetzel
MSVC Error:
2>D:\zgh\Libs\suitesparse\SuiteSparse\CHOLMOD\Supernodal\../GPU/t_cholmod_gpu.c(55,9): error C3016: 'i': index variable in OpenMP 'for' statement must have signed integral type

[mmuetzel]

Thank you for testing.
See also this similar issue where it seems to have been solved by adding /openmp:llvm to the compiler flags:
tesseract-ocr/tesseract#3285

Does that also work for you?

[mmuetzel]

Looking at the affected code snippet again: num_chunks is already of type int. So, i can be of the same size.

Your change looks good to me. 👍

[DrTimothyAldenDavis]

Looks good -- thanks.