[llvm-commits] [llvm] r155364 - in /llvm/trunk: include/llvm/CodeGen/DFAPacketizer.h lib/CodeGen/DFAPacketizer.cpp

[Sirish Pande]

There were only two reviews:

1. Eric C asked for test cases, and that test cases be associated with 
the patch.
2. Tom Stellard had requested to split the vliw packetizer patch.

Both have been done.

On top that, Andrew had some changes in the Scheduler which had to be incorporated.

Other than that it's the same patch. Why do you doubt that these patches are different?

Sirish


On 4/23/2012 1:01 PM, Chandler Carruth wrote:
> On Mon, Apr 23, 2012 at 10:49 AM, Sirish Pande <spande at codeaurora.org 
> <mailto:spande at codeaurora.org>> wrote:
>
>     Author: sirish
>     Date: Mon Apr 23 12:49:09 2012
>     New Revision: 155364
>
>     URL: http://llvm.org/viewvc/llvm-project?rev=155364&view=rev
>     <http://llvm.org/viewvc/llvm-project?rev=155364&view=rev>
>     Log:
>     Hexagon Packetizer's target independent fix.
>
>
> Did you receive off-list review for this that you haven't mentioned here?
>
> I strongly doubt it as these patches were mailed in their final form 
> on Saturday of last week, and you have submitted them fairly early on 
> Monday morning.


-- 
Qualcomm Innovation Center, Inc is a member of Code Aurora Forum

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120423/839f313e/attachment.html>