<div dir="ltr">r226710</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 21, 2015 at 2:33 PM, Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">----- Original Message -----<br>
> From: "George Burgess IV" <<a href="mailto:george.burgess.iv@gmail.com">george.burgess.iv@gmail.com</a>><br>
</span><span class="">> To: "Hal J. Finkel" <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>><br>
> Cc: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> Sent: Wednesday, January 21, 2015 11:36:18 AM<br>
> Subject: Re: [llvm] r226671 - Fixed a bug with how we determine bitset indices.<br>
><br>
> Diff containing only the test case is attached — I’ll start including<br>
> test cases for changes like this in the future. :)<br>
<br>
</span>Thanks!<br>
<span class=""><br>
><br>
> Is the crash-the-compiler style of test (given the circumstances)<br>
> acceptable, or do you think it would be best to verify the<br>
> compiler’s output as well?<br>
<br>
</span>Verifying the output is preferred, so long as the output is expected to be stable relative to what is being checked. It is not required, however. In this case, checking the output seems straightforward, so you might as well do it. Regardless, LGTM.<br>
<br>
 -Hal<br>
<br>
><br>
> George<br>
<div class="HOEnZb"><div class="h5">><br>
><br>
><br>
><br>
> > On Jan 21, 2015, at 11:42 AM, Hal Finkel <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>> wrote:<br>
> ><br>
> > Do you have a test case?<br>
> ><br>
> > -Hal<br>
> ><br>
> > ----- Original Message -----<br>
> >> From: "George Burgess IV" <<a href="mailto:george.burgess.iv@gmail.com">george.burgess.iv@gmail.com</a>><br>
> >> To: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> >> Sent: Wednesday, January 21, 2015 10:37:21 AM<br>
> >> Subject: [llvm] r226671 - Fixed a bug with how we determine bitset<br>
> >> indices.<br>
> >><br>
> >> Author: gbiv<br>
> >> Date: Wed Jan 21 10:37:21 2015<br>
> >> New Revision: 226671<br>
> >><br>
> >> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=226671&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=226671&view=rev</a><br>
> >> Log:<br>
> >> Fixed a bug with how we determine bitset indices.<br>
> >><br>
> >><br>
> >> Modified:<br>
> >> llvm/trunk/lib/Analysis/CFLAliasAnalysis.cpp<br>
> >><br>
> >> Modified: llvm/trunk/lib/Analysis/CFLAliasAnalysis.cpp<br>
> >> URL:<br>
> >> <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/CFLAliasAnalysis.cpp?rev=226671&r1=226670&r2=226671&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/CFLAliasAnalysis.cpp?rev=226671&r1=226670&r2=226671&view=diff</a><br>
> >> ==============================================================================<br>
> >> --- llvm/trunk/lib/Analysis/CFLAliasAnalysis.cpp (original)<br>
> >> +++ llvm/trunk/lib/Analysis/CFLAliasAnalysis.cpp Wed Jan 21<br>
> >> 10:37:21<br>
> >> 2015<br>
> >> @@ -774,7 +774,7 @@ static Optional<StratifiedAttr> valueToA<br>
> >> }<br>
> >><br>
> >> static StratifiedAttr argNumberToAttrIndex(unsigned ArgNum) {<br>
> >> - if (ArgNum > AttrMaxNumArgs)<br>
> >> + if (ArgNum >= AttrMaxNumArgs)<br>
> >> return AttrAllIndex;<br>
> >> return ArgNum + AttrFirstArgIndex;<br>
> >> }<br>
> >><br>
> >><br>
> >> _______________________________________________<br>
> >> llvm-commits mailing list<br>
> >> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> >> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
> >><br>
> ><br>
> > --<br>
> > Hal Finkel<br>
> > Assistant Computational Scientist<br>
> > Leadership Computing Facility<br>
> > Argonne National Laboratory<br>
><br>
><br>
<br>
--<br>
Hal Finkel<br>
Assistant Computational Scientist<br>
Leadership Computing Facility<br>
Argonne National Laboratory<br>
</div></div></blockquote></div><br></div>