[clang] [clang] Improve diagnostics for `__builtin_align` builtins with floating/member pointer operands (PR #192650)

[Erich Keane via cfe-commits]

=?utf-8?b?8J+NjFNoYXdu?= <m18824909883 at 163.com>,
=?utf-8?b?8J+NjFNoYXdu?= <m18824909883 at 163.com>,
=?utf-8?b?8J+NjFNoYXdu?= <m18824909883 at 163.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/192650 at github.com>


================
@@ -336,10 +336,12 @@ static bool BuiltinAlignment(Sema &S, CallExpr *TheCall, unsigned ID) {
   }
   if ((!SrcTy->isPointerType() && !IsValidIntegerType(SrcTy)) ||
       SrcTy->isFunctionPointerType()) {
----------------
erichkeane wrote:

Do we have a diagnostic we could add for the `isFunctionPointerType` here as well?  It would be nice to finish out the 'list' of diagnostics. 

https://github.com/llvm/llvm-project/pull/192650