This is an archive of the discontinued LLVM Phabricator instance.

[OPENMP]Fix PR48394: need to capture variables used in atomic constructs.
ClosedPublic

Authored by ABataev on Dec 4 2020, 1:00 PM.

Details

Summary

The variables used in atomic construct should be captured in outer
task-based regions implicitly. Otherwise, the compiler will crash trying
to find the address of the local variable.

Diff Detail

Event Timeline

ABataev created this revision.Dec 4 2020, 1:00 PM
ABataev requested review of this revision.Dec 4 2020, 1:00 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 4 2020, 1:00 PM
Herald added a subscriber: sstefan1. · View Herald Transcript
jdoerfert accepted this revision.Dec 4 2020, 1:07 PM

LGTM, thanks for the quick fix!

This revision is now accepted and ready to land.Dec 4 2020, 1:07 PM
This revision was landed with ongoing or failed builds.Dec 4 2020, 1:40 PM
This revision was automatically updated to reflect the committed changes.