[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