Skip to content

[PAC][Driver] Choose between two ways of implementing ptrauth flags #97320

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
kovdan01 opened this issue Jul 1, 2024 · 2 comments
Open

[PAC][Driver] Choose between two ways of implementing ptrauth flags #97320

kovdan01 opened this issue Jul 1, 2024 · 2 comments
Labels
clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl'

Comments

@kovdan01
Copy link
Contributor

kovdan01 commented Jul 1, 2024

Currently, we have a bunch of -fptrauth-* flags. As discussed in #96160 (comment), we might want to change -fptrauth-xxx -fptrauth-yyy to smth like -fptrauth=xxx,yyy.

This issue is intended to be used as a thread for discussion whether we need to implement such a change. Personally, I like the way with comma-separated options, but I have no issues with current state of flags as well.

@kovdan01
Copy link
Contributor Author

kovdan01 commented Jul 1, 2024

Tagging @MaskRay @ahmedbougacha @asl

@EugeneZelenko EugeneZelenko added clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl' and removed new issue labels Jul 1, 2024
@llvmbot
Copy link
Member

llvmbot commented Jul 1, 2024

@llvm/issue-subscribers-clang-driver

Author: Daniil Kovalev (kovdan01)

Currently, we have a bunch of `-fptrauth-*` flags. As discussed in https://github.com//pull/96160#issuecomment-2183305495, we might want to change `-fptrauth-xxx -fptrauth-yyy` to smth like `-fptrauth=xxx,yyy`.

This issue is intended to be used as a thread for discussion whether we need to implement such a change. Personally, I like the way with comma-separated options, but I have no issues with current state of flags as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl'
Projects
Status: No status
Development

No branches or pull requests

3 participants