Skip to content
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

[clang-cl] /Fe option with colon is not accepted #46065

Closed
andreyv opened this issue Jul 15, 2020 · 1 comment
Closed

[clang-cl] /Fe option with colon is not accepted #46065

andreyv opened this issue Jul 15, 2020 · 1 comment
Labels
bugzilla Issues migrated from bugzilla clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl'

Comments

@andreyv
Copy link
Member

andreyv commented Jul 15, 2020

Bugzilla Link 46720
Resolution FIXED
Resolved on Jul 20, 2020 03:08
Version 10.0
OS Windows NT
CC @zmodem,@zygoloid

Extended Description

According to [1], /Fe option has two forms - it can be used with : as a separator.

Currently clang-cl.exe supports only /Fe (without :)

Steps to reproduce:

clang-cl.exe "/Fe:out2.exe" test.cpp

Does not produce out2.exe (produces no executable at all)

Whereas

clang-cl.exe "/Feout.exe" test.cpp

works fine.

[1] https://docs.microsoft.com/en-us/cpp/build/reference/fe-name-exe-file

@zmodem
Copy link
Collaborator

zmodem commented Jul 20, 2020

Thanks! Fixed in 8513a68

I wonder if this form is new. Looking at the docs, the other /F options don't seem to have this variant.

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 10, 2021
rnk pushed a commit that referenced this issue Apr 1, 2024
Modeled after
8513a68

According to
https://learn.microsoft.com/en-us/cpp/build/reference/fo-object-file-name?view=msvc-170,
`/Fo` accepts a trailing-colon variant. This is also tested in practice.
This allows clang-cl to parse this.

I just copied one of the existing tests, let me know if this is not the
best way to do this. I tested that the test does not pass beofre the
Options.td change, and that it does after.

See also #46065
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl'
Projects
None yet
Development

No branches or pull requests

2 participants