This is an archive of the discontinued LLVM Phabricator instance.

[Attributor] Change AAExecutionDomain to only accept intrinsics
ClosedPublic

Authored by jhuber6 on May 18 2021, 9:57 AM.

Details

Summary

The OpenMP runtime functions don't always provide unique thread ID's to
determine if a basic block is truly single-threaded. Change the implementation
to only check NVPTX intrinsics for now.

Diff Detail

Event Timeline

jhuber6 created this revision.May 18 2021, 9:57 AM
jhuber6 requested review of this revision.May 18 2021, 9:57 AM
Herald added a reviewer: baziotis. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
jdoerfert accepted this revision.May 18 2021, 11:35 AM

LG, can you please add the AMDGPU intrinsic as well (with test)

This revision is now accepted and ready to land.May 18 2021, 11:35 AM
jhuber6 updated this revision to Diff 346262.May 18 2021, 1:38 PM

Adding AMDGCN intrinsic check.

This revision was landed with ongoing or failed builds.May 18 2021, 6:32 PM
This revision was automatically updated to reflect the committed changes.