[PATCH 1/2] TableGen: Generate a function for getting operand indices based on their defined names

Jakob Stoklund Olesen stoklund at 2pi.dk
Thu Jun 13 07:41:17 PDT 2013


On Jun 7, 2013, at 4:30 PM, Tom Stellard <tom at stellard.net> wrote:

> On Fri, Jun 07, 2013 at 09:48:55AM -0700, Jakob Stoklund Olesen wrote:
>> 
>> The function you’re generating looks like it could get really big. Please use some form of table compression to reduce the size.
>> 
> 
> Hi Jakob,
> 
> Here are updated patches that incorporate your comments.
> 
> -Tom
> <0001-TableGen-Generate-a-function-for-getting-operand-ind.patch><0002-R600-Use-new-getNamedOperandIdx-function-generated-b.patch>

Tom, please include code size numbers, now that we are talking about code size.

>From a quick look at your code, those tables still look pretty big to me.

/jakob





More information about the llvm-commits mailing list