This is an archive of the discontinued LLVM Phabricator instance.

feat: Add support for Rocky Linux to LLVM Distro
Needs RevisionPublic

Authored by zkkxu on Jul 28 2023, 11:15 PM.

Diff Detail

Event Timeline

zkkxu created this revision.Jul 28 2023, 11:15 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 28 2023, 11:15 PM
zkkxu requested review of this revision.Jul 28 2023, 11:15 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 28 2023, 11:15 PM

@MaskRay Isn't this something we should be doing with config files now?

zkkxu updated this revision to Diff 545358.Jul 29 2023, 3:43 AM
  • fix wrong ut test
This comment was removed by zkkxu.
zkkxu added a comment.Jul 31 2023, 7:23 AM

nice ping.

MaskRay added a comment.EditedJul 31 2023, 1:25 PM

@MaskRay Isn't this something we should be doing with config files now?

I agree. This should use https://clang.llvm.org/docs/UsersManual.html#configuration-files .

clang/include/clang/Driver/Distro.h is not a place to enumerate every distribution. It provides some methods e.g. IsOpenSuse to be checked by driver code to add, for example, default linker options for a particular distribution.
However, this has been largely replaced with https://clang.llvm.org/docs/UsersManual.html#configuration-files and we should migrate some to use configuration files.

https://blogs.gentoo.org/mgorny/2022/10/07/clang-in-gentoo-now-sets-default-runtimes-via-config-file/ has some nice information.

MaskRay requested changes to this revision.Jul 31 2023, 1:26 PM
This revision now requires changes to proceed.Jul 31 2023, 1:26 PM