Re: [PATCH][pushed] if-to-switch: Fix test-suite patterns.

2020-12-02 Thread Richard Biener via Gcc-patches
On Tue, Dec 1, 2020 at 9:04 PM Martin Liška wrote: > > On 12/1/20 8:57 PM, Jakub Jelinek wrote: > > Would be better to use \[^\n\r]\* instead of .*, with .* you can match parts > > even from completely different parts of the dump file. > > Thank you Jakub! I've just pushed a patch with that. Mayb

Re: [PATCH][pushed] if-to-switch: Fix test-suite patterns.

2020-12-01 Thread Martin Liška
On 12/1/20 8:57 PM, Jakub Jelinek wrote: Would be better to use \[^\n\r]\* instead of .*, with .* you can match parts even from completely different parts of the dump file. Thank you Jakub! I've just pushed a patch with that. Martin

Re: [PATCH][pushed] if-to-switch: Fix test-suite patterns.

2020-12-01 Thread Jakub Jelinek via Gcc-patches
On Tue, Dec 01, 2020 at 08:52:45PM +0100, Martin Liška wrote: > -/* { dg-final { scan-tree-dump "Condition chain with 3 BBs transformed into > a switch statement." "iftoswitch" } } */ > +/* { dg-final { scan-tree-dump "Condition chain with .* BBs transformed into > a switch statement." "iftoswitc

[PATCH][pushed] if-to-switch: Fix test-suite patterns.

2020-12-01 Thread Martin Liška
On ppc64le some conditions in BBs are not merged so we merge a different number of BBs. Pushed as obvious. Tested on x86_64-linux-gnu and ppc64le-linux-gnu. gcc/testsuite/ChangeLog: PR testsuite/98085 * g++.dg/tree-ssa/if-to-switch-1.C: Do not expect precise number of BB