erichkeane added a comment.

In https://reviews.llvm.org/D36272#855957, @anatol.pomozov wrote:

> The function-attributes.c test  does not pass at Windows 
> <https://reviews.llvm.org/D36272#851708> even without my patch. Unfortunately 
> I am not familiar with clang enough to debug this issue.


Yes, but the point of your patch seems to be to ADD this functionality to x64 
Linux AND windows.
My question is what happens with:
-triple i386-pc-win32?

This needs to match the behavior of 
-triple x86_64-pc-win32


https://reviews.llvm.org/D36272



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to