[clang] [clang] assume_aligned incorrectly diagnoses a dependent return type (PR #111573)

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 8 11:38:26 PDT 2024


================
@@ -1216,6 +1216,8 @@ static void handlePreferredName(Sema &S, Decl *D, const ParsedAttr &AL) {
 }
 
 bool Sema::isValidPointerAttrType(QualType T, bool RefOkay) {
+  if (T->isDependentType())
----------------
erichkeane wrote:

Probably valuable to audit the other uses of this function well, and remove checks for dependence there as redundant.  And probably ensure the documentation of this function mentions that it skips on dependence.

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


More information about the cfe-commits mailing list