[PATCH] Warn on suspicious use of absolute value function

Richard Trieu rtrieu at google.com
Wed Feb 12 14:22:05 PST 2014


  Before suggesting a different absolute value function, check the namespace to see if it is visible.  If it doesn't exist, also give a note with the header to include.

http://llvm-reviews.chandlerc.com/D2224

CHANGE SINCE LAST DIFF
  http://llvm-reviews.chandlerc.com/D2224?vs=6591&id=7034#toc

Files:
  lib/Sema/SemaChecking.cpp
  test/Sema/warn-absolute-value.c
  test/Sema/warn-absolute-value-header.c
  test/SemaCXX/warn-absolute-value-header.cpp
  include/clang/Basic/DiagnosticSemaKinds.td
  include/clang/Basic/DiagnosticGroups.td
  include/clang/Sema/Sema.h
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D2224.3.patch
Type: text/x-patch
Size: 62818 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140212/abed0175/attachment.bin>


More information about the cfe-commits mailing list