[cfe-commits] Patch: Add warning for undefined reinterpret_cast behavior

Argyrios Kyrtzidis kyrtzidis at apple.com
Mon May 2 11:33:57 PDT 2011

On Apr 29, 2011, at 7:48 PM, Richard T wrote:

> Moved the logic for checking undefined behavior and its warnings into CheckCompatibleReinterpretCast().  Also, cleaned up the test case a little.

Thanks! Submitted in r130703 with a tweak; I changed

+  if (Diags.getDiagnosticLevel(DiagID, SourceLocation()) ==
+          Diagnostic::Ignored) {


+  if (Diags.getDiagnosticLevel(DiagID, Range.getBegin()) ==
+          Diagnostic::Ignored) {

FYI, it's important to call Diags.getDiagnosticLevel with the location where you intend to emit the warning; in the presence of diagnostic pragmas it is possible that a small section in the source has the warning enabled while it is normally disabled.

Chandler, if you can evaluate the new warning that'd be great.

More information about the cfe-commits mailing list