> 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