Re: [PATCH] SSA ALIAS: Apply LEN_MASK_STORE to 'ref_maybe_used_by_call_p_1'

2023-06-24 Thread Jeff Law via Gcc-patches
On 6/23/23 17:20, 钟居哲 wrote: Not sure since I saw MASK_STORE/LEN_STORE didn't compute size. Also OK after a re-review on my part. The code sets the size to -1 after doing the ao_ref_init_from_ptr_and_size, meaning it's not a known size. jeff

Re: Re: [PATCH] SSA ALIAS: Apply LEN_MASK_STORE to 'ref_maybe_used_by_call_p_1'

2023-06-23 Thread 钟居哲
Not sure since I saw MASK_STORE/LEN_STORE didn't compute size. juzhe.zh...@rivai.ai From: Jeff Law Date: 2023-06-24 00:27 To: juzhe.zhong; gcc-patches CC: rguenther; richard.sandiford Subject: Re: [PATCH] SSA ALIAS: Apply LEN_MASK_STORE to 'ref_maybe_used_by_call_p_1' On 6/23/23 08:15

Re: [PATCH] SSA ALIAS: Apply LEN_MASK_STORE to 'ref_maybe_used_by_call_p_1'

2023-06-23 Thread Jeff Law via Gcc-patches
On 6/23/23 08:15, juzhe.zh...@rivai.ai wrote: From: Ju-Zhe Zhong gcc/ChangeLog: * tree-ssa-alias.cc (call_may_clobber_ref_p_1): Add LEN_MASK_STORE. Doesn't this need to extract/compute the size argument in a manner similar to what DSE does? Jeff

[PATCH] SSA ALIAS: Apply LEN_MASK_STORE to 'ref_maybe_used_by_call_p_1'

2023-06-23 Thread juzhe . zhong
From: Ju-Zhe Zhong gcc/ChangeLog: * tree-ssa-alias.cc (call_may_clobber_ref_p_1): Add LEN_MASK_STORE. --- gcc/tree-ssa-alias.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/gcc/tree-ssa-alias.cc b/gcc/tree-ssa-alias.cc index 92dc1bb9987..f31fd042c2a 100644 ---