[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