<div class="gmail_quote">On Wed, Jan 11, 2012 at 12:19 AM, Abramo Bagnara <span dir="ltr"><<a href="mailto:abramo.bagnara@gmail.com">abramo.bagnara@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: abramo<br>
Date: Wed Jan 11 02:19:46 2012<br>
New Revision: 147933<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=147933&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=147933&view=rev</a><br>
Log:<br>
Fixed VLA code generation.<br></blockquote><div><br></div><div>In general, it would help a lot to be more descriptive about what went wrong previously in commit logs. It's often not obvious from the patch what the prior misbehavior was, and having that context makes it faster to review these changes.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Added:<br>
    cfe/trunk/test/CodeGen/vla-4.c<br></blockquote><div><br></div><div>Also, rather than having now 4 test files for vlas, it would be better to add these test cases and their checks to existing files, and potentially even collapse the files we already have.</div>
</div>