diff options
author | Argyrios Kyrtzidis <akyrtzi@gmail.com> | 2010-11-17 18:26:36 +0000 |
---|---|---|
committer | Argyrios Kyrtzidis <akyrtzi@gmail.com> | 2010-11-17 18:26:36 +0000 |
commit | 14a96625b870c7a20dc1a9cea44a45d65e76863b (patch) | |
tree | 04d0a99a8438b77ad21095575626e3115edb2f1f /llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp | |
parent | f10c43c7f791938c65005b0a605723bc9f240a74 (diff) | |
download | bcm5719-llvm-14a96625b870c7a20dc1a9cea44a45d65e76863b.tar.gz bcm5719-llvm-14a96625b870c7a20dc1a9cea44a45d65e76863b.zip |
Don't warn for parentheses for the '&&' inside '||' for cases like:
assert(a || b && "bad");
since this is safe. This way we avoid a big source of such warnings which in this case are practically useless.
Note that we don't handle *all* cases where precedence wouldn't matter because of constants since
this is a bit costly to check, and IMO clarifying precedence with parentheses is good for
readability in general.
llvm-svn: 119533
Diffstat (limited to 'llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp')
0 files changed, 0 insertions, 0 deletions