This is an archive of the discontinued LLVM Phabricator instance.

[CUDA, FDO] Filter out profiling options from GPU-side compilations.
ClosedPublic

Authored by tra on Apr 15 2021, 1:54 PM.

Diff Detail

Event Timeline

tra created this revision.Apr 15 2021, 1:54 PM
tra requested review of this revision.Apr 15 2021, 1:54 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 15 2021, 1:54 PM
tra added inline comments.Apr 15 2021, 1:56 PM
clang/lib/Driver/ToolChains/Clang.cpp
5828–5848

@yaxunl Should I add isHipDevice here too? I don't think host-side profiling info is of much use for GPU compilation.

yaxunl added inline comments.Apr 15 2021, 2:07 PM
clang/lib/Driver/ToolChains/Clang.cpp
5828–5848

Yes pls do the same for HIP. Thanks.

tra updated this revision to Diff 337896.Apr 15 2021, 2:12 PM

filter the options for AMD GPUs too.

tra marked an inline comment as done.Apr 15 2021, 2:13 PM
yaxunl accepted this revision.Apr 15 2021, 2:32 PM

LGTM. Thanks!

This revision is now accepted and ready to land.Apr 15 2021, 2:32 PM