Target Dependent Hexagon Packetizer patch

Project Infrastructure LLVM Dev List Archives
#1

Hi,

Here's a patch for Hexagon Packetizer for review. This patch does not yield any warnings.

Sirish

HexagonPacketizer.patch (445 KB)

#2

No test cases for a 500k patch?

-eric

#3

Hi,

Here's a patch for Hexagon Packetizer for review. This patch does
not yield any warnings.

Would it be possible to split this patch in two? One for the core
LLVM changes and one for the Hexagon changes. I've been working on a
Packetizer for the R600 backend, and I was using the DFAPacketizer code
that was reverted, so it would be nice if the DFAPacketizer changes
weren't tied directly to the Hexagon code.

-Tom

#4

No, not at this time, but we will be putting them over the time.

Sirish

#5

I am working on some tests for Hexagon and I will put them sometime today.

Sirish

#6

Sure I will split it and put it in two patches.

Give me few hours. I need to test those patches.

Sirish