Re: [PATCH] waccess: Look at calls when tracking clobbers [PR104092]

2022-02-04 Thread Richard Sandiford via Gcc-patches
Richard Sandiford writes: > In this PR the waccess pass was fed: > > D.10779 ={v} {CLOBBER}; > VIEW_CONVERT_EXPR(D.10779) = .MASK_LOAD_LANES (addr_5(D), > 64B, _2); > _7 = D.10779.__val[0]; > > However, the tracking of m_clobbers only looked at gassigns, > so it missed that the clobber on t

Re: [PATCH] waccess: Look at calls when tracking clobbers [PR104092]

2022-01-19 Thread Martin Sebor via Gcc-patches
On 1/19/22 09:22, Richard Sandiford wrote: Martin Sebor writes: On 1/19/22 03:09, Richard Sandiford wrote: Richard Biener writes: On Tue, Jan 18, 2022 at 2:40 PM Richard Sandiford via Gcc-patches wrote: In this PR the waccess pass was fed: D.10779 ={v} {CLOBBER}; VIEW_CONVERT_EXP

Re: [PATCH] waccess: Look at calls when tracking clobbers [PR104092]

2022-01-19 Thread Richard Sandiford via Gcc-patches
Martin Sebor writes: > On 1/19/22 03:09, Richard Sandiford wrote: >> Richard Biener writes: >>> On Tue, Jan 18, 2022 at 2:40 PM Richard Sandiford via Gcc-patches >>> wrote: In this PR the waccess pass was fed: D.10779 ={v} {CLOBBER}; VIEW_CONVERT_EXPR(D.10779) = .M

Re: [PATCH] waccess: Look at calls when tracking clobbers [PR104092]

2022-01-19 Thread Martin Sebor via Gcc-patches
On 1/19/22 03:09, Richard Sandiford wrote: Richard Biener writes: On Tue, Jan 18, 2022 at 2:40 PM Richard Sandiford via Gcc-patches wrote: In this PR the waccess pass was fed: D.10779 ={v} {CLOBBER}; VIEW_CONVERT_EXPR(D.10779) = .MASK_LOAD_LANES (addr_5(D), 64B, _2); _7 = D.10779.

Re: [PATCH] waccess: Look at calls when tracking clobbers [PR104092]

2022-01-19 Thread Richard Sandiford via Gcc-patches
Richard Biener writes: > On Tue, Jan 18, 2022 at 2:40 PM Richard Sandiford via Gcc-patches > wrote: >> >> In this PR the waccess pass was fed: >> >> D.10779 ={v} {CLOBBER}; >> VIEW_CONVERT_EXPR(D.10779) = .MASK_LOAD_LANES (addr_5(D), >> 64B, _2); >> _7 = D.10779.__val[0]; >> >> However, th

Re: [PATCH] waccess: Look at calls when tracking clobbers [PR104092]

2022-01-19 Thread Richard Biener via Gcc-patches
On Tue, Jan 18, 2022 at 2:40 PM Richard Sandiford via Gcc-patches wrote: > > In this PR the waccess pass was fed: > > D.10779 ={v} {CLOBBER}; > VIEW_CONVERT_EXPR(D.10779) = .MASK_LOAD_LANES (addr_5(D), > 64B, _2); > _7 = D.10779.__val[0]; > > However, the tracking of m_clobbers only looked

Re: [PATCH] waccess: Look at calls when tracking clobbers [PR104092]

2022-01-18 Thread Jason Merrill via Gcc-patches
On 1/18/22 13:06, Martin Sebor wrote: On 1/18/22 06:37, Richard Sandiford wrote: In this PR the waccess pass was fed:    D.10779 ={v} {CLOBBER};    VIEW_CONVERT_EXPR(D.10779) = .MASK_LOAD_LANES (addr_5(D), 64B, _2);    _7 = D.10779.__val[0]; However, the tracking of m_clobbers only looked at

Re: [PATCH] waccess: Look at calls when tracking clobbers [PR104092]

2022-01-18 Thread Martin Sebor via Gcc-patches
On 1/18/22 06:37, Richard Sandiford wrote: In this PR the waccess pass was fed: D.10779 ={v} {CLOBBER}; VIEW_CONVERT_EXPR(D.10779) = .MASK_LOAD_LANES (addr_5(D), 64B, _2); _7 = D.10779.__val[0]; However, the tracking of m_clobbers only looked at gassigns, so it missed that the clobber

[PATCH] waccess: Look at calls when tracking clobbers [PR104092]

2022-01-18 Thread Richard Sandiford via Gcc-patches
In this PR the waccess pass was fed: D.10779 ={v} {CLOBBER}; VIEW_CONVERT_EXPR(D.10779) = .MASK_LOAD_LANES (addr_5(D), 64B, _2); _7 = D.10779.__val[0]; However, the tracking of m_clobbers only looked at gassigns, so it missed that the clobber on the first line was overwritten by the call o