[Mesa-dev] [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 mesa-dev
mailing list