Page MenuHomePhabricator

Cleanup of affinity hierarchy code.
ClosedPublic

Authored by jlpeyton on Aug 28 2015, 2:59 PM.

Details

Summary

Some of this is improvement to code suggested by Hal Finkel. Four changes here:

  1. Cleanup of hierarchy code to handle all hierarchy cases whether affinity is available or not
  2. Separated this and other classes and common functions out to a header file
  3. Added a destructor-like fini function for the hierarchy (and call in __kmp_cleanup)
  4. Remove some redundant code that is hopefully no longer needed

Diff Detail

Event Timeline

jlpeyton updated this revision to Diff 33472.Aug 28 2015, 2:59 PM
jlpeyton retitled this revision from to Cleanup of affinity hierarchy code..
jlpeyton updated this object.
jlpeyton added reviewers: AndreyChurbanov, hfinkel.
jlpeyton set the repository for this revision to rL LLVM.
jlpeyton added a subscriber: openmp-commits.
AndreyChurbanov accepted this revision.Sep 8 2015, 9:14 AM
AndreyChurbanov edited edge metadata.

LGTM

This revision is now accepted and ready to land.Sep 8 2015, 9:14 AM
This revision was automatically updated to reflect the committed changes.