Skip to content

Commit

Permalink
delete [-Wno-error=terminate], test=develop (#52490)
Browse files Browse the repository at this point in the history
* delete [-Wno-error=terminate], test=develop

* remove GPUps[-Wterminate],test=develop
  • Loading branch information
Galaxy1458 authored Apr 4, 2023
1 parent 80a4a2e commit 15aa73d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
1 change: 0 additions & 1 deletion cmake/flags.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,6 @@ if(NOT WIN32)
-Wno-error=literal-suffix
-Wno-error=array-bounds #Warning in Eigen, gcc 12.2
-Wno-error=ignored-attributes # Warnings in Eigen, gcc 6.3
-Wno-error=terminate # Warning in PADDLE_ENFORCE
-Wno-error=int-in-bool-context # Warning in Eigen gcc 7.2
-Wimplicit-fallthrough=0 # Warning in tinyformat.h
${fsanitize})
Expand Down
6 changes: 3 additions & 3 deletions paddle/fluid/framework/fleet/heter_ps/heter_resource.cc
Original file line number Diff line number Diff line change
Expand Up @@ -55,13 +55,13 @@ GPUResource::GPUResource(std::vector<int> &dev_ids, int index) {
GPUResource::~GPUResource() {
platform::CUDADeviceGuard guard(dev_id_);
for (size_t i = 0; i < local_streams_.size(); ++i) {
PADDLE_ENFORCE_GPU_SUCCESS(cudaStreamDestroy(local_streams_[i]));
PADDLE_WARN_GPU_SUCCESS(cudaStreamDestroy(local_streams_[i]));
}
for (size_t i = 0; i < comm_streams_.size(); ++i) {
PADDLE_ENFORCE_GPU_SUCCESS(cudaStreamDestroy(comm_streams_[i]));
PADDLE_WARN_GPU_SUCCESS(cudaStreamDestroy(comm_streams_[i]));
}
for (size_t i = 0; i < remote_streams_.size(); ++i) {
PADDLE_ENFORCE_GPU_SUCCESS(cudaStreamDestroy(remote_streams_[i]));
PADDLE_WARN_GPU_SUCCESS(cudaStreamDestroy(remote_streams_[i]));
}
}

Expand Down

0 comments on commit 15aa73d

Please sign in to comment.