Page MenuHomePhabricator
Differential D96892

[OpenMP] libomp: cleanup some resource leaks
ClosedPublic
Actions

Authored by AndreyChurbanov on Feb 17 2021, 1:05 PM.

Details

Reviewers
hbae
jlpeyton
tlwilmar
Nawrin
jdoerfert
Commits
rG1611e5473c5e: [OpenMP] libomp: cleanup some resource leaks
Summary

Close mutexattr and condattr local objects to eliminate resource leaks.

Diff Detail

Event Timeline

AndreyChurbanov created this revision.Feb 17 2021, 1:05 PM
Herald added subscribers: guansong, yaxunl. · View Herald TranscriptFeb 17 2021, 1:05 PM
AndreyChurbanov requested review of this revision.Feb 17 2021, 1:05 PM
Herald added a reviewer: jdoerfert. · View Herald TranscriptFeb 17 2021, 1:05 PM
Herald added subscribers: openmp-commits, sstefan1. · View Herald Transcript
Harbormaster completed remote builds in B89600: Diff 324399.Feb 17 2021, 2:29 PM
Nawrin accepted this revision.Feb 19 2021, 3:22 PM

LGTM

This revision is now accepted and ready to land.Feb 19 2021, 3:22 PM
Closed by commit rG1611e5473c5e: [OpenMP] libomp: cleanup some resource leaks (authored by AndreyChurbanov). · Explain WhyFeb 20 2021, 12:28 PM
This revision was automatically updated to reflect the committed changes.
AndreyChurbanov added a commit: rG1611e5473c5e: [OpenMP] libomp: cleanup some resource leaks.

Revision Contents
Changeset List

PathSize
openmp/
runtime/
src/
4 lines

Diff 325230

View Options

openmp/runtime/src/z_Linux_util.cpp

Loading...