LegalizeAdulthood added a comment.

I didn't know about the bug reports.  I created this check because I have 
encountered such redundant control flow in real code bases.

This would close 21984 <https://llvm.org/bugs/show_bug.cgi?id=21984>, but not 
22416 <https://llvm.org/bugs/show_bug.cgi?id=22416>.  22416 feels more like a 
check to eliminate redundant parenthesis, whereas this check is concerned with 
the entire control flow statement being redundant.


http://reviews.llvm.org/D16259



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to