[PATCH] R600: Remove AMDILPeeopholeOptimizer and replace optimizations with tablegen patterns

Tom Stellard tom at stellard.net
Wed May 8 06:58:17 PDT 2013


On Tue, May 07, 2013 at 10:48:53AM +0200, Christian König wrote:
> Am 06.05.2013 18:48, schrieb Tom Stellard:
> >From: Tom Stellard <thomas.stellard at amd.com>
> >
> >The BFE optimization was the only one we were actually using, and it was
> >emitting an intrinsic that we don't support.
> >
> >https://bugs.freedesktop.org/show_bug.cgi?id=64201
> 
> The patch has my rb.
> 
> I'm wondering if we shouldn't get ride of all those AMDIL* files,
> cause we obviously don't use that namespace anymore and at least I
> sometimes wonder why we still have AMDILISelLowering.cpp and
> AMDGPUISelLowering.cpp.
>

I would like to drop the AMDIL* files.  Most of them are still used
in one way or another, so maybe all we can do now is rename them.

I left AMDILISelLowering.cpp in because I was worried removing it would
break things, but maybe it's safe to remove it now that the backend is
more mature.

-Tom




More information about the llvm-commits mailing list