Skip to content

Conversation

@nguidotti
Copy link
Contributor

Main branch fails to compile due to the maybe-uninitialized error in the bytes variable in the linear_programming/utils.cuh file (see the log below), which is fixed by this PR.

[1/3] Building CUDA object CMakeFiles/cuopt.dir/src/linear_programming/pdlp.cu.o
FAILED: [code=1] CMakeFiles/cuopt.dir/src/linear_programming/pdlp.cu.o 
ccache /home/nguidotti/miniconda/envs/cuopt_dev/bin/nvcc -forward-unknown-to-host-compiler -DCCCL_DISABLE_PDL -DCUB_DISABLE_NAMESPACE_MAGIC -DCUB_IGNORE_NAMESPACE_MAGIC_ERROR -DCUOPT_CPU_ARCHITECTURE=\"x86_64\" -DCUOPT_CUDA_ARCHITECTURES=\"90a-real\" -DCUOPT_GIT_COMMIT_HASH=\"326eec6\" -DCUOPT_LOG_ACTIVE_LEVEL=RAPIDS_LOGGER_LOG_LEVEL_INFO -DCUTLASS_NAMESPACE=raft_cutlass -DLIBCUDACXX_ENABLE_EXPERIMENTAL_MEMORY_RESOURCE -DNVTX_ENABLED -DRAFT_LOG_ACTIVE_LEVEL=RAPIDS_LOGGER_LOG_LEVEL_INFO -DRAFT_SYSTEM_LITTLE_ENDIAN=1 -DTHRUST_DEVICE_SYSTEM=THRUST_DEVICE_SYSTEM_CUDA -DTHRUST_DISABLE_ABI_NAMESPACE -DTHRUST_HOST_SYSTEM=THRUST_HOST_SYSTEM_CPP -DTHRUST_IGNORE_ABI_NAMESPACE_ERROR -Dcuopt_EXPORTS -I/cuopt-dev/cuopt/cpp/../thirdparty -I/cuopt-dev/cuopt/cpp/src -I/cuopt-dev/cuopt/cpp/include -I/cuopt-dev/cuopt/cpp/build/include -I/cuopt-dev/cuopt/cpp/libmps_parser/include -I/cuopt-dev/cuopt/cpp/build/_deps/cccl-src/lib/cmake/thrust/../../../thrust -I/cuopt-dev/cuopt/cpp/build/_deps/cccl-src/lib/cmake/libcudacxx/../../../libcudacxx/include -I/cuopt-dev/cuopt/cpp/build/_deps/cccl-src/lib/cmake/cub/../../../cub -isystem /cuopt-dev/cuopt/cpp/build/_deps/papilo-src/src -isystem /cuopt-dev/cuopt/cpp/build/_deps/papilo-build -isystem /home/nguidotti/miniconda/envs/cuopt_dev/targets/x86_64-linux/include -isystem /home/nguidotti/miniconda/envs/cuopt_dev/targets/x86_64-linux/include/cccl --expt-relaxed-constexpr --expt-extended-lambda -static-global-template-stub=false -O3 -DNDEBUG -lineinfo -std=c++17 "--generate-code=arch=compute_90a,code=[sm_90a]" -Xcompiler=-fPIC -Werror=cross-execution-space-call -Wno-deprecated-declarations -Xcompiler=-Werror -Xcompiler=-Wall -Wno-error=non-template-friend -Xfatbin=-compress-all -fopenmp -lineinfo --expt-extended-lambda --expt-relaxed-constexpr -MD -MT CMakeFiles/cuopt.dir/src/linear_programming/pdlp.cu.o -MF CMakeFiles/cuopt.dir/src/linear_programming/pdlp.cu.o.d -x cu -c /cuopt-dev/cuopt/cpp/src/linear_programming/pdlp.cu -o CMakeFiles/cuopt.dir/src/linear_programming/pdlp.cu.o
/cuopt-dev/cuopt/cpp/src/linear_programming/utils.cuh: In function 'void cuopt::linear_programming::detail::compute_sum_bounds(const rmm::device_uvector<i_t>&, const rmm::device_uvector<i_t>&, rmm::device_scalar<i_t>&, rmm::cuda_stream_view) [with f_t = double]':
/cuopt-dev/cuopt/cpp/src/linear_programming/utils.cuh:225:22: error: 'bytes' may be used uninitialized [-Werror=maybe-uninitialized]
  225 |   d_temp_storage.resize(bytes, stream_view);
      |   ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~
/cuopt-dev/cuopt/cpp/src/linear_programming/utils.cuh:205:8: note: 'bytes' was declared here
  205 |   size_t bytes;
      |        ^~~~~
cc1plus: all warnings being treated as errors
ninja: build stopped: subcommand failed.

@nguidotti nguidotti requested a review from a team as a code owner October 1, 2025 11:49
@nguidotti nguidotti requested review from Kh4ster, aliceb-nv and rg20 and removed request for rg20 October 1, 2025 11:49
@nguidotti nguidotti added bug Something isn't working non-breaking Introduces a non-breaking change labels Oct 1, 2025
@nguidotti nguidotti requested review from akifcorduk and removed request for Kh4ster and aliceb-nv October 1, 2025 12:02
@nguidotti
Copy link
Contributor Author

/ok to test b909bd0

@akifcorduk
Copy link
Contributor

/merge

@rapids-bot rapids-bot bot merged commit 60a9816 into NVIDIA:branch-25.10 Oct 2, 2025
288 of 298 checks passed
@nguidotti nguidotti deleted the fix-pdlp-compilation branch October 9, 2025 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants