[clang] [Clang] Add __datasizeof (PR #67805)

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 13 11:49:53 PDT 2023


================
@@ -1016,4 +1016,7 @@ def warn_unpacked_field
 def warn_unaligned_access : Warning<
   "field %1 within %0 is less aligned than %2 and is usually due to %0 being "
   "packed, which can lead to unaligned accesses">, InGroup<UnalignedAccess>, DefaultIgnore;
+
+def err_cannot_mangle_expression : Error<
+  "cannot yet mangle %0 expression">;
----------------
AaronBallman wrote:

Ah, so the "obvious" may be more obvious to people used to working with Clang's diagnostics and not so obvious to folks not as well-versed. Good feedback!

I'm okay with the changes but I have a preference for doing this in two steps. Step 1: your patch w/custom diagnostics engine, Step 2: replace all custom diagnostics with the regular diagnostic in one go. (This makes code archeology easier because the changes are kind of orthogonal to one another.) Would you be okay with splitting it up despite the extra work?

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


More information about the cfe-commits mailing list