On Mon, Mar 17, 2014 at 2:22 PM, Richard Biener <richard.guent...@gmail.com> wrote: > On Sun, Mar 16, 2014 at 1:21 PM, Prathamesh Kulkarni > <bilbotheelffri...@gmail.com> wrote: >> In c_expr::c_expr, shouldn't OP_C_EXPR be passed to operand >> constructor instead of OP_EXPR ? > > Indeed - I have committed the fix. > My earlier mail got rejected (maybe because I attached pdf ?), by mailer daemon, sorry for the double post. I have uploaded the proposal here: https://drive.google.com/file/d/0B7zFk-y3DFiHa1Nkdzh6TFZpVFE/edit?usp=sharing I would be grateful to receive your feedback.
Thanks and Regards, Prathamesh > Thanks, > Richard. > >> This caused segfault for patterns when "simplification" operand was >> only c_expr (patch attached). >> >> * genmatch.c (c_expr::c_expr): use OP_C_EXPR instead of OP_EXPR in >> call to operand constructor.
