This is an archive of the discontinued LLVM Phabricator instance.

[libomptarget] Move resource id functions into target specific code, implement for amdgcn
ClosedPublic

Authored by JonChesterfield on Dec 11 2019, 5:57 PM.

Details

Summary

[libomptarget] Move resource id functions into target specific code, implement for amdgcn

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptDec 11 2019, 5:57 PM
JonChesterfield added a comment.EditedDec 11 2019, 6:00 PM

Had some trouble picking a name for this file. It's a place to put the four accesses to magic cuda variables so that non-cuda targets can provide an equivalent, with its own header as that seemed better than implementing support.h across multiple sources.

ABataev added inline comments.Dec 12 2019, 9:06 AM
openmp/libomptarget/deviceRTLs/amdgcn/src/resource_id.hip
1

We have target-specific files already, like target_impl.h, maybe just put all these stuff in this file?

JonChesterfield marked an inline comment as done.Dec 12 2019, 9:45 AM
JonChesterfield added inline comments.
openmp/libomptarget/deviceRTLs/amdgcn/src/resource_id.hip
1

target_impl is becoming a bit of a dumping ground, but sure - another four functions won't hurt

  • Fold resource_id into target_impl as requested
  • remove whitespace from comment
JonChesterfield marked an inline comment as done.Dec 12 2019, 9:53 AM
Harbormaster completed remote builds in B42410: Diff 233651.
This revision is now accepted and ready to land.Dec 12 2019, 10:49 AM
This revision was automatically updated to reflect the committed changes.