efriedma added inline comments.

================
Comment at: lib/AST/ASTContext.cpp:2088
+  default:
+    UnadjustedAlign = getTypeAlign(T);
+  }
----------------
This "default" isn't right; there are a lot of non-canonical types which need 
to be handled here (which getTypeAlign handles, but this doesn't).

Could you just write something like `if (const auto *RTy = 
T->getAs<RecordType>()) [...] else return getTypeAlign(T);`?


https://reviews.llvm.org/D46013



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

Reply via email to