[PATCH] R600/SI: Fix crash in SIInstrInfo::areLoadsFromSameBasePtr()
Matt Arsenault
arsenm2 at gmail.com
Fri Mar 20 18:27:35 PDT 2015
> On Mar 20, 2015, at 5:15 PM, Tom Stellard <thomas.stellard at amd.com> wrote:
>
> This function assumed that SMRD instructions always have immediate
> offsets, which is not always the case.
> ---
> lib/Target/R600/SIInstrInfo.cpp | 12 +++++++--
> test/CodeGen/R600/schedule-kernel-arg-loads.ll | 35 ++++++++++++++++++++++++--
> 2 files changed, 43 insertions(+), 4 deletions(-)
>
LGTM. How did you run into this? I thought this was dead code since the mi scheduler was enabled
More information about the llvm-commits
mailing list