[COMMITTED] Re: [PATCH] PR tree-optimization/119039 - Simplify switches utilizing subranges.

2025-06-18 Thread Andrew MacLeod
On 6/18/25 14:17, Andrew Pinski wrote: On Wed, Jun 18, 2025, 11:11 AM Andrew MacLeod wrote: On 6/18/25 14:00, Andrew Pinski wrote: > > > +/* { dg-final { scan-tree-dump-not "foo" "evrp" } } */ > > May I suggest this be changed to "foo " (adding a space). To avoid

Re: [PATCH] PR tree-optimization/119039 - Simplify switches utilizing subranges.

2025-06-18 Thread Andrew Pinski
On Wed, Jun 18, 2025, 11:11 AM Andrew MacLeod wrote: > > On 6/18/25 14:00, Andrew Pinski wrote: > > > > > > +/* { dg-final { scan-tree-dump-not "foo" "evrp" } } */ > > > > May I suggest this be changed to "foo " (adding a space). To avoid > > maybe matching on a directory name that contains foo.

Re: [PATCH] PR tree-optimization/119039 - Simplify switches utilizing subranges.

2025-06-18 Thread Andrew MacLeod
On 6/18/25 14:00, Andrew Pinski wrote: +/* { dg-final { scan-tree-dump-not "foo" "evrp" } } */ May I suggest this be changed to "foo " (adding a space). To avoid maybe matching on a directory name that contains foo. I only noticed this today after you pushed thr change sorry. Thanks, And

Re: [PATCH] PR tree-optimization/119039 - Simplify switches utilizing subranges.

2025-06-18 Thread Andrew Pinski
On Tue, Jun 10, 2025, 10:03 AM Andrew MacLeod wrote: > simplify_switch_using_ranges has never been converted over properly to > use irange. It still uses get_legacy_range, and as a result, is very > limited in how it processes cases labels. In particular it is unable to > remove cases in the mi

Re: [PATCH] PR tree-optimization/119039 - Simplify switches utilizing subranges.

2025-06-18 Thread Andrew MacLeod
Pushed.   Lets see if the thread testcase triggers a different number of threads somewhere else. Andrew On 6/10/25 13:01, Andrew MacLeod wrote: simplify_switch_using_ranges has never been converted over properly to use irange.  It still uses get_legacy_range, and as a result, is very limited

[PATCH] PR tree-optimization/119039 - Simplify switches utilizing subranges.

2025-06-10 Thread Andrew MacLeod
simplify_switch_using_ranges has never been converted over properly to use irange.  It still uses get_legacy_range, and as a result, is very limited in how it processes cases labels.  In particular it is unable to remove cases in the middle which are not possible to call. This patch rewrites t