Hi,
On Wed, Jan 10, 2018 at 01:55:34PM -0600, Peter Bergner wrote:
> >> @@ -570,7 +570,7 @@ (define_split
> >> ;; The post-reload split requires that we re-permute the source
> >> ;; register in case it is still live.
> >> (define_split
> >> - [(set (match_operand:VSX_D 0 "memory_operand" "")
> >> + [(set (match_operand:VSX_D 0 "indexed_or_indirect_operand" "")
> >> (match_operand:VSX_D 1 "vsx_register_operand" ""))]
> >> "!BYTES_BIG_ENDIAN && TARGET_VSX && !TARGET_P9_VECTOR &&
> >> reload_completed"
> >> [(set (match_dup 1)
> >
> > You don't mention these in the changelog.
>
> I tried to by mentioning splitters in the entry above. How are these
> unnamed splitters supposed to be mentioned? Maybe like:
>
> (*vsx_le_perm_store_<mode> for <VSX_D>): Likewise/
> (*vsx_le_perm_store_<mode> for <VSX_D> splitter): Likewise.
They don't have that name (they don't have any name).
I often say things like
(8 unnamed splitters): Likewise.
(you can try to merge a define_split first, to a define_insn_and_split,
which is a nice cleanup by itself; but that won't work here I think).
Segher
