This is an archive of the discontinued LLVM Phabricator instance.

[clang][Sema][OpenMP] Fix capture region for 'target teams loop'
Needs ReviewPublic

Authored by ddpagan on Aug 4 2023, 2:16 PM.

Details

Reviewers
ABataev
jdoerfert
Summary

The capture region for 'target teams loop' should be the same as that for 'target teams distribute parallel for' with an 'if(target:val)'. For this
specific case, there should be no capture region.

Updated test results for OpenMP/target_teams_generic_loop_if_codegen.cpp

Diff Detail

Event Timeline

ddpagan created this revision.Aug 4 2023, 2:16 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 4 2023, 2:16 PM
ddpagan requested review of this revision.Aug 4 2023, 2:16 PM
ABataev added inline comments.Aug 5 2023, 2:01 PM
clang/lib/Sema/SemaOpenMP.cpp
15410

Need to have capture region for parallel, no?