[PATCH] Fix R0 use in PowerPC VSX store for FastIsel

Samuel Antao sfantao at us.ibm.com
Mon Mar 16 11:19:21 PDT 2015


Before this fix, there were instructions being generated like this:

STXSDX %http://reviews.llvm.org/F1<kill>, %X3<kill>, %noreg, %RM<imp-use>

The code seems be prepared to handle %noreg. R0 was being emitted for %noreg. The problem here is not the register class but the order of the operands. If FastISel were emitting something like:

STXSDX %http://reviews.llvm.org/F1<kill>, %X3<kill>, %ZERO8, %RM<imp-use>

we would still have a problem, as the content of R0 would be taken to compute the effective address of the store. If we have:

STXSDX %http://reviews.llvm.org/F1<kill>, %ZERO8, %X3<kill>, %RM<imp-use>

this works fine as R0 used as based address makes 0 to be used in the computation of the effective address instead of the value contained in R0. We are doing something similar for when UseOffset is set, the immediate has to come before the base register in order to get the right semantics.

Thanks,
Samuel


http://reviews.llvm.org/D8358

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list