| commit | 7cabe6d207a3a2fbfd2f02f838fa51bc469a5fe7 | |
| author | segher <segher@138bc75d-0d04-0410-961f-82ee72b054a4> | |
| Wed, 22 Feb 2017 23:50:46 +0000 (22 23:50 +0000) | ||
| committer | segher <segher@138bc75d-0d04-0410-961f-82ee72b054a4> | |
| Wed, 22 Feb 2017 23:50:46 +0000 (22 23:50 +0000) | ||
| tree | 0d88fc2c67e4724582485e7125a2c65df904e471 | treesnapshot (tar.gz zip) |
| parent | 1f075b0e389c3454dae00437e16ffb4c08d065fe | commitdiff |
rs6000: Fix fsel pattern (PR79211)
The fsel define_insn uses fpr_reg_operand for its predicates. This
won't work because passes can put a hard register in the operands: in
the testcase, combine likes to forward the parameter registers to what
then is still an smin, and then split1 uses "*s<minmax><mode>3_fpr"
(which has gpc_reg_operand). And then we have a GPR in the operand,
which does not match fpr_reg_operand.
It seems to me the predicates should be gpc_reg_operand here as well.
This patch changes that.
PR target/79211
* config/rs6000/rs6000.md (*fsel<SFDF:mode><SFDF2:mode>4): Use
gpc_reg_operand instead of fpr_reg_operand.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@245667 138bc75d-0d04-0410-961f-82ee72b054a4
The fsel define_insn uses fpr_reg_operand for its predicates. This
won't work because passes can put a hard register in the operands: in
the testcase, combine likes to forward the parameter registers to what
then is still an smin, and then split1 uses "*s<minmax><mode>3_fpr"
(which has gpc_reg_operand). And then we have a GPR in the operand,
which does not match fpr_reg_operand.
It seems to me the predicates should be gpc_reg_operand here as well.
This patch changes that.
PR target/79211
* config/rs6000/rs6000.md (*fsel<SFDF:mode><SFDF2:mode>4): Use
gpc_reg_operand instead of fpr_reg_operand.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@245667 138bc75d-0d04-0410-961f-82ee72b054a4
| gcc/ChangeLog | diffblobblamehistory | |
| gcc/config/rs6000/rs6000.md | diffblobblamehistory |