[PATCH] D46805: If some platforms do not support an attribute, we should exclude the platform

[Dan Gohman via Phabricator via cfe-commits]

sunfish added a comment.

In https://reviews.llvm.org/D46805#1115681, @rsmith wrote:

> In https://reviews.llvm.org/D46805#1113358, @aaron.ballman wrote:
>
> > @rsmith -- do the object file formats listed look correct to you?
>
>
> They look at least plausible. We should be able to test whether LLVM can actually emit aliases on each of these targets easily enough...
>
> However, I get this error for any WAsm compilation I try:
>
>   fatal error: error in backend: section size does not fit in a uint32_t
>
> ... so I have no idea if aliases are/will be supported there. Perhaps @sunfish can tell us :)


Yes, they are intended to be supported. It sounds like you found a bug, though I've not been able to reproduce it in simple tests.


Repository:
  rC Clang

https://reviews.llvm.org/D46805