On Fri, Sep 28, 2012 at 06:24:13PM +0200, Michel Dänzer wrote:
> From: Michel Dänzer <michel.daen...@amd.com>
>
> This reverts commit bfd55711c1eb32ffbfceb9d566abae98f0015f23.
>
> It makes piglit unreliable due to VM protection faults and GPU lockups.
>
> Signed-off-by: Michel Dänzer <michel.daen...@amd.com>
> src/gallium/drivers/radeon/AMDGPUTargetMachine.cpp | 1 -
>
> diff --git a/src/gallium/drivers/radeon/AMDGPUTargetMachine.cpp
> b/src/gallium/drivers/radeon/AMDGPUTargetMachine.cpp
> index 0b70191..c1b6840 100644
> --- a/src/gallium/drivers/radeon/AMDGPUTargetMachine.cpp
> +++ b/src/gallium/drivers/radeon/AMDGPUTargetMachine.cpp
> @@ -134,7 +134,6 @@ bool AMDGPUPassConfig::addPreEmitPass() {
> addPass(FinalizeMachineBundlesID);
> } else {
> PM->add(createSILowerLiteralConstantsPass(*TM));
> - PM->add(createSILowerFlowControlPass(*TM));
> }
>
Instead of reverting the entire commit, you can just tell the
TargetMachine not to add the SILowerFlowControlPass. It should have the
same effect as a full revert.
With that change:
Reviewed-by: Tom Stellard <thomas.stell...@amd.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
