[llvm-commits] [PATCH] Add Pattern ID Information

David A. Greene greened at obbligato.org
Thu Aug 4 13:48:46 PDT 2011


Bruno Cardoso Lopes <bruno.cardoso at gmail.com> writes:

>> The refactoring will make things more maintainable now and prepare for
>> the future.  We talked about this a year ago and folks seemed to agree
>> it was a good idea.
>
> I agree with Chris, we don't wanna bloat the actual descriptions and
> special C++ handling, based on a forecast, specially because it will
> include code that isn't tested nor have any testcases for.

It doesn't bloat.  It reduces.

And it's not a forecast.  This stuff is well-known already.  It's here
today.  Look at how similar AVX is to SSE and how similar SSE is to MMX.
There's tons of redundancy in the existing specification.  It's
non-orthogonal and cumbersome to maintain.

Honestly, we really don't want to prepare for the future and make things
simpler for the average developer in the process?  Especially after the
discussion about a year or so ago where people seemed at least
interested in the idea?

At least allow me to present the work without facing a crowd that's
already made up its mind.

And I'm still waiting on a patch review.  If no one has objections I
guess I will start committing.  I'll wait until tomorrow though.

                                 -Dave




More information about the llvm-commits mailing list