lebedev.ri added a comment.

Nice!

  /home/eenitot/new_llvm/llvm/lib/Support/ConvertUTFWrapper.cpp:38:26: warning: 
do not cast pointers into more strictly aligned pointer types [cert-exp36-c]
      UTF16 *targetStart = reinterpret_cast<UTF16*>(ResultPtr);
                           ^

Hmm, the text is well-worded, but i wonder if a bit more context, much like the 
`-Wcast-align`, would be nicer, something like:

  warning: casting from type `T1` (alignment `a1`) to type `T2` (alignment 
`a2`) increases required alignment [cert-exp36-c]

It does mix the alignment of the type `T` and the alignment specified by 
`__attribute__((aligned()))` / `alignas()`, but i'm not sure it is possible to 
show that in the message..
Thoughts?


Repository:
  rL LLVM

https://reviews.llvm.org/D33826



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

Reply via email to