WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED INVALID
72582
WTF_EXPORT should be clang aware.
https://bugs.webkit.org/show_bug.cgi?id=72582
Summary
WTF_EXPORT should be clang aware.
Hajime Morrita
Reported
2011-11-17 00:19:51 PST
Curretly WTF_EXPORT doesn't care about Clang compiler. Because Clang understands the same "visibility" attribute as gcc, we can let the macro definition to allow it under compilation with Clang.
Attachments
Patch
(1.31 KB, patch)
2011-11-17 00:22 PST
,
Hajime Morrita
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Hajime Morrita
Comment 1
2011-11-17 00:22:39 PST
Created
attachment 115539
[details]
Patch
Simon Hausmann
Comment 2
2011-11-17 00:29:04 PST
Comment on
attachment 115539
[details]
Patch Hm, I think the best way to write this expression is: #elif COMPILER(GCC) || COMPILER(CLANG) #COMPILER(GCC) implies __GNUC__ && !COMPILER(RVCT) and COMPILER(RVCT) is __CC_ARM or __ARMCC__ However I'm somewhat surprised CLANG doesn't identify itself as gcc (for compatibility).
Hajime Morrita
Comment 3
2011-11-17 00:34:19 PST
> However I'm somewhat surprised CLANG doesn't identify itself as gcc (for compatibility).
Whoa, good point. It actually knows __GNUC__. This patch is invalid.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug