compnerd added a comment.

I think that it makes more sense to ignore the `aapcs-vfp` calling convention 
attribute (returning `CCCR_Ignore`), and warn on the `aapcs`.  The redundant 
calling convention attribute does nothing, and I think that simply ignoring it 
would solve your issue.  However, changing the calling convention to `aapcs` 
would break interoperability, so warning on that seems reasonable still.  I 
think that if Wine is willing to adjust their use of the attribute, that would 
really be nicer - it allows us to keep the behaviour as close to MSVC as 
possible.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94364/new/

https://reviews.llvm.org/D94364

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

Reply via email to