RE: [PATCH v3 2/5] Match: Add interface match_cond_with_binary_phi for true/false arg

2024-09-11 Thread Li, Pan2
ject: Re: [PATCH v3 2/5] Match: Add interface match_cond_with_binary_phi for true/false arg On Thu, Sep 12, 2024 at 3:41 AM Li, Pan2 wrote: > > Thanks Richard for comments. > > > why would arg_edge depend on whether t0 is INTEGER_CST or not? > Because the edge->src of INTEGER_CST

Re: [PATCH v3 2/5] Match: Add interface match_cond_with_binary_phi for true/false arg

2024-09-11 Thread Richard Biener
ou are thinking of, more complex CFGs should always mean more than a single controlling condition - I'm not sure we want to go the way to present those as cond1 | cond2. Richard. > Pan > > -Original Message----- > From: Richard Biener > Sent: Wednesday, September 11, 2024 9:39 PM

RE: [PATCH v3 2/5] Match: Add interface match_cond_with_binary_phi for true/false arg

2024-09-11 Thread Li, Pan2
:39 PM To: Li, Pan2 Cc: gcc-patches@gcc.gnu.org; tamar.christ...@arm.com; juzhe.zh...@rivai.ai; kito.ch...@gmail.com; jeffreya...@gmail.com; rdapp....@gmail.com Subject: Re: [PATCH v3 2/5] Match: Add interface match_cond_with_binary_phi for true/false arg On Wed, Sep 11, 2024 at 8:31 AM wrote: &

Re: [PATCH v3 2/5] Match: Add interface match_cond_with_binary_phi for true/false arg

2024-09-11 Thread Richard Biener
On Wed, Sep 11, 2024 at 8:31 AM wrote: > > From: Pan Li > > When matching the cond with 2 args phi node, we need to figure out > which arg of phi node comes from the true edge of cond block, as > well as the false edge. This patch would like to add interface > to perform the action and return th

[PATCH v3 2/5] Match: Add interface match_cond_with_binary_phi for true/false arg

2024-09-10 Thread pan2 . li
From: Pan Li When matching the cond with 2 args phi node, we need to figure out which arg of phi node comes from the true edge of cond block, as well as the false edge. This patch would like to add interface to perform the action and return the true and false arg in TREE type. There will be som