This is an archive of the discontinued LLVM Phabricator instance.

[Clang] Remove `CLANG_OPENMP_NVPTX_DEFAULT_ARCH` CMake option.
ClosedPublic

Authored by jhuber6 on Jan 13 2023, 1:04 PM.

Details

Summary

The CLANG_OPENMP_NVPTX_DEFAULT_ARCH is a static build configuration to
set the default OpenMP value. This was replaced in D141708 with the use
of the nvptx-arch tool which lets us query this at runtime instead.
This makes the behaviour between AMD and NVIDIA be more consisten and
allows users to have a default architecture that does not rely on
whoever configured the LLVM build.

Depends on D141708

Diff Detail

Event Timeline

jhuber6 created this revision.Jan 13 2023, 1:04 PM
jhuber6 requested review of this revision.Jan 13 2023, 1:04 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJan 13 2023, 1:04 PM

Up right you will find Edit Related Revisions....

MaskRay accepted this revision.Jan 13 2023, 1:19 PM

Removing a CMAKE_*_DEFAULT_* makes me happy:)

This revision is now accepted and ready to land.Jan 13 2023, 1:19 PM
tra accepted this revision.Jan 13 2023, 1:52 PM
This revision was landed with ongoing or failed builds.Jan 13 2023, 2:52 PM
This revision was automatically updated to reflect the committed changes.