Skip to content

In microsoft, support __fastcall on functions without prototype #13080

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

Closed
nico opened this issue Apr 30, 2012 · 5 comments
Closed

In microsoft, support __fastcall on functions without prototype #13080

nico opened this issue Apr 30, 2012 · 5 comments
Labels
bugzilla Issues migrated from bugzilla clang:frontend Language frontend issues, e.g. anything involving "Sema" wontfix Issue is real, but we can't or won't fix it. Not invalid

Comments

@nico
Copy link
Contributor

nico commented Apr 30, 2012

Bugzilla Link 12708
Resolution WONTFIX
Resolved on Jul 26, 2014 14:11
Version unspecified
OS Windows NT
Blocks llvm/llvm-bugzilla-archive#13707
CC @asl,@zmodem,@tritao

Extended Description

handleFunctionTypeAttr in SemaType.cpp

C:\src\chrome\src>type test.c
void __fastcall f() {
}

C:\src\chrome\src>cl /c test.c
Microsoft (R) 32-bit C/C++ Optimizing Compiler Version 16.00.30319.01 for 80x86
Copyright (C) Microsoft Corporation. All rights reserved.

test.c

C:\src\chrome\src>....\llvm-build\bin\Release\clang.exe -c test.c
test.c:1:6: error: function with no prototype cannot use fastcall calling convention
void __fastcall f() {
^
1 error generated.

This is used in lzma_sdk if _MSC_VER is defined.

@tritao
Copy link
Mannequin

tritao mannequin commented Jun 21, 2012

Can't reproduce this on trunk. Is it fixed?

@nico
Copy link
Contributor Author

nico commented Jun 23, 2012

I still see this with today's trunk (r159059). It only happens for .c files, not .cc.

@tritao
Copy link
Mannequin

tritao mannequin commented Jun 23, 2012

Ah you are right. I didn't notice it only happens on .c files.

@nico
Copy link
Contributor Author

nico commented Jul 26, 2014

See comment 2 on issue 20386 for why this isn't a good idea.

@tritao
Copy link
Mannequin

tritao mannequin commented Nov 26, 2021

mentioned in issue llvm/llvm-bugzilla-archive#13707

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 3, 2021
@Quuxplusone Quuxplusone added the wontfix Issue is real, but we can't or won't fix it. Not invalid label Jan 20, 2022
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:frontend Language frontend issues, e.g. anything involving "Sema" wontfix Issue is real, but we can't or won't fix it. Not invalid
Projects
None yet
Development

No branches or pull requests

2 participants