On 12/14/2015 03:20 PM, Richard Biener wrote:
But there must be a reason statics/externals are expected and handled in all the callers (and the function).
At the time this was written (git rev 60d031234), expand_one_var looked very different. It called a subroutine expand_one_static_var which did things like call rest_of_decl_compilation. That stuff was then removed by Jan in a patch to drop non-unit-at-a-time compilation.
The new early-out just makes the code even more confusing than before.
Ok, so let's bail out even earlier. The whole expand_now thing for statics is likely to be just a historical artifact at this point.
Bernd
