On 05/24/12 09:03, Richard Guenther wrote:
If you don't hear from somebody else in 24h the patch is ok as-is (can you do some grepping whether there are callers of build_constructor that set TREE_SIDE_EFFECTS on it afterwards?)
I committed the patch. grepping C & C++ sources didn't show callers of build_constructor poking T_S_E themselves. some of them did clear TREE_CONSTANT though. Rather than chase the rationale for that I decided to let them lie.
nathan