<div dir="ltr">So remove the ifStmt from the third and fourth case? <div><br></div><div>So that I keep if(str1.compare(str2)) and if(!str1.compare(str2)), and change the other two to str1.compare(str2) == 0 and str1.compare(str2) != 0 ? </div><div><br></div><div>That makes good sense. Then I could also add some of the test cases you mentioned earlier.</div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 30, 2016 at 5:59 PM Malcolm Parsons via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">malcolm.parsons added a comment.<br class="gmail_msg">
<br class="gmail_msg">
I don't know why you're restricting this check to only match within the condition of an if statement.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D27210" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D27210</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>