[cfe-commits] [PATCH] Handle __declspec(dllimport) and __declspec(dllexport)

Charles Davis cdavis at mymail.mines.edu
Sun Feb 14 15:02:42 PST 2010


Anton Korobeynikov wrote:
> Hello, Charles
> 
>> appropriately. As far as I know, these are the only two declspec
>> attributes that clang supports at the moment. It fixes the reporter's
>> immediate problem, but I wouldn't call the bug "fixed" because of the
>> many, many MS declspec attributes we don't support yet.
> Right. That's why it make sense to move the handling of these
> attributes to TargetSema (possible slightly extending this interface).
> Could you please consider this way?
Yeah, I could do that.

The problem is that these aren't x86-specific attributes. They're
Windows/Symbian-specific attributes. So now, the way I envision it, we
need another class hierarchy for OS-specific attributes as opposed to
architecture-specific attributes. (In actuality, I'll probably create
another subclass of TargetAttributesSema for this instance, and then add
a new method, getOSTargetAttributesSema() to get something from this new
family.)

Chip




More information about the cfe-commits mailing list