diff options
author | David Blaikie <dblaikie@gmail.com> | 2012-05-15 21:57:38 +0000 |
---|---|---|
committer | David Blaikie <dblaikie@gmail.com> | 2012-05-15 21:57:38 +0000 |
commit | 18e9ac79146bd6740b92472d73aa252db314e83b (patch) | |
tree | 13da8c5e78295c25c9e57dc46f5836848a1ceb6c /llvm/tools/llvm-config/llvm-config.cpp | |
parent | edd124ec72c2b4b433245dc9d5264feb06017b36 (diff) | |
download | bcm5719-llvm-18e9ac79146bd6740b92472d73aa252db314e83b.tar.gz bcm5719-llvm-18e9ac79146bd6740b92472d73aa252db314e83b.zip |
Don't warn when NULL is used within a macro but its conversion is outside a macro.
This fixes the included test case & was reported by Nico Weber.
It's a little bit nasty using the difference in the conversion context, but
seems to me like a not unreasonable solution. I did have to fix up the
conversion context for conditional operators (it seems correct to me to include
the context for which we're actually doing the comparison - across all the
nested conditionals, rather than the innermost conditional which might not
actually have the problematic implicit conversion at all) and template default
arguments (this is a bit of a hack, since we don't have the source location of
the '=' anymore, so I just used the start of the parameter - open to
suggestions there)
llvm-svn: 156861
Diffstat (limited to 'llvm/tools/llvm-config/llvm-config.cpp')
0 files changed, 0 insertions, 0 deletions