[PATCH] D24098: AMDGPU/SI: Don't run SLSR and NaryReassociate passes

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 31 13:43:56 PDT 2016


arsenm added a comment.

The bug should be fixed instead of this. However I noticed this is run for -O0 which should be fixed

In https://reviews.llvm.org/D24098#530751, @tstellarAMD wrote:

> In https://reviews.llvm.org/D24098#530742, @arsenm wrote:
>
> > The bug should be fixed instead of this. However I noticed this is run for -O0 which should be fixed
>
>
> I agree, but these bugs are preventing us from being able to compiler some programs, and I'm not sure when someone will be able to look into fixing this bug.


With just ripping this out, it's more likely that nobody will ever get around to re-enabling these. How about just adding a flag to allow disabling this so those programs can work around it for now? Alternatively this bug should be higher priority


https://reviews.llvm.org/D24098





More information about the llvm-commits mailing list