This is an archive of the discontinued LLVM Phabricator instance.

__kmp_partition_places: Update assertion for new parameter
ClosedPublic

Authored by Hahnfeld on Jul 1 2016, 4:29 AM.

Details

Summary

If update_master_only is set the place list is not completely traversed and therefore this assertion failed. Make it only trigger if update_master_only is false.

(was introduced by D20539)

Diff Detail

Repository
rL LLVM

Event Timeline

Hahnfeld updated this revision to Diff 62481.Jul 1 2016, 4:29 AM
Hahnfeld retitled this revision from to __kmp_partition_places: Update assertion for new parameter.
Hahnfeld updated this object.
Hahnfeld added reviewers: jlpeyton, tlwilmar.
Hahnfeld added a subscriber: openmp-commits.
This revision is now accepted and ready to land.Jul 1 2016, 10:06 AM
jlpeyton accepted this revision.Jul 1 2016, 11:02 AM
jlpeyton edited edge metadata.

LGTM too.

This revision was automatically updated to reflect the committed changes.