lkml.org 
[lkml]   [2020]   [Feb]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/4] perf expr: Move expr lexer to flex
On Fri, Feb 21, 2020 at 04:23:12PM -0800, Andi Kleen wrote:
> > +max { return MAX; }
> > +min { return MIN; }
> > +if { return IF; }
> > +else { return ELSE; }
> > +#smt_on { return SMT_ON; }
> > +{number} { return value(yyscanner, 10); }
> > +{symbol} { return str(yyscanner, ID); }
> > +"|" { return '|'; }
> > +"^" { return '^'; }
> > +"&" { return '&'; }
> > +"-" { return '-'; }
> > +"+" { return '+'; }
> > +"*" { return '*'; }
> > +"/" { return '/'; }
> > +"%" { return '%'; }
> > +"(" { return '('; }
> > +")" { return ')'; }
> > +"," { return ','; }
>
> Didn't think there was a comma, but ok.

actually it's for the min/max functions arguments separation
I posted v2

jirka

>
> Looks reasonable to me.
>
> -Andi
>

\
 
 \ /
  Last update: 2020-02-24 09:30    [W:0.043 / U:1.024 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site