[Bug target/69120] sse2_shufpd_v2df_mask has wrong name

2016-02-03 Thread kyukhin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69120

--- Comment #2 from Kirill Yukhin  ---
Looked closely.
The name was chosen intentionally to simplify "sse2_shufpd"
expand. If we want to fix this name - new subst attribute need to be introduced
and 
if () 
  emit_insn (avx512vl_...
else
  emit_insn (sse2_...
inserted into the expand.

Beside of the expand this template never called by name.

So, I bet to have the name unchanged and keep things simple.

[Bug target/69120] sse2_shufpd_v2df_mask has wrong name

2016-02-03 Thread kyukhin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69120

--- Comment #1 from Kirill Yukhin  ---
Will fix.

[Bug target/69120] sse2_shufpd_v2df_mask has wrong name

2016-02-03 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69120

--- Comment #3 from H.J. Lu  ---
Can you add a comment and resolve it?

[Bug target/69120] sse2_shufpd_v2df_mask has wrong name

2016-02-03 Thread kyukhin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69120

Kirill Yukhin  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |WONTFIX

--- Comment #4 from Kirill Yukhin  ---
This makes corresponding expand much simpler. So I think this little naming
inconsistency is preferable to additional checks in define_expand.

[Bug target/69120] sse2_shufpd_v2df_mask has wrong name

2016-01-03 Thread kyukhin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69120

Kirill Yukhin  changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2016-01-03
 CC||kyukhin at gcc dot gnu.org
   Assignee|unassigned at gcc dot gnu.org  |kyukhin at gcc dot 
gnu.org
 Ever confirmed|0   |1