[PATCH] D28271: [Sema] Get rid of unused default argument to Sema::CheckCallingConvAttr.

[Reid Kleckner via Phabricator via cfe-commits]

rnk added a comment.

I think you've discovered this bug:

  struct F { void __attribute__((pcs("aapcs"))) f(); };
  void __attribute__((pcs("aapcs"))) F::f() {}

Clang ignores the pcs convention on 32-bit x86, and because we don't have an FD 
here, we choose the wrong default calling convention (not thiscall) for F::f.


https://reviews.llvm.org/D28271



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