[Bug target/91016] A suspicious code in config/i386/i386-expand.c

2019-06-27 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91016 Martin Liška changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug target/91016] A suspicious code in config/i386/i386-expand.c

2019-06-27 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91016 --- Comment #5 from Martin Liška --- Author: marxin Date: Thu Jun 27 11:22:36 2019 New Revision: 272746 URL: https://gcc.gnu.org/viewcvs?rev=272746&root=gcc&view=rev Log: Fix various issues seen with clang-static-analyzer. 2019-06-27 Martin Li

[Bug target/91016] A suspicious code in config/i386/i386-expand.c

2019-06-27 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91016 --- Comment #4 from Uroš Bizjak --- (In reply to Martin Liška from comment #3) > (In reply to Uroš Bizjak from comment #2) > > ix86_expand_sse_compare_mask generates temporary by itself, so the REG RTX > > is just overwritten. There is no issue w

[Bug target/91016] A suspicious code in config/i386/i386-expand.c

2019-06-27 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91016 Martin Liška changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Assignee|unassigned at

[Bug target/91016] A suspicious code in config/i386/i386-expand.c

2019-06-27 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91016 --- Comment #2 from Uroš Bizjak --- ix86_expand_sse_compare_mask generates temporary by itself, so the REG RTX is just overwritten. There is no issue with reusage of pseudos as they are short lived, and die in AND RTX anyway. OTOH, non-AVX AND RT

[Bug target/91016] A suspicious code in config/i386/i386-expand.c

2019-06-27 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91016 --- Comment #1 from Martin Liška --- Jakub's comment from https://gcc.gnu.org/ml/gcc-patches/2019-06/msg01659.html: Just quick look suggests perhaps it wants to use one temporary for the gen_reg_rtx (mode) and use that on the lhs of SET, and ano