> I've installed it on trunk but will give it quite a while there before
> backporting.  I'm still somewhat worried about the
> 
>       /* ???  If we end up with a constant or a descriptor do not
>          record a MEM_EXPR.  */
>       else if (CONSTANT_CLASS_P (t)
> 
>                || TREE_CODE (t) == CONSTRUCTOR)
> 
>         ;
> 
> case, maybe we should assert that bitpos == 0 here since we're
> dropping it on the floor but eventually inherit offset_known_p == true
> from the already present MEM_ATTRs.

It's hard to imagine without a testcase how you can end up inheriting anything 
for CONSTANT_CLASS_P or CONSTRUCTOR though: where do the MEM_ATTRs come from? 

-- 
Eric Botcazou

Reply via email to