[incremental] Patch: FYI: more late gimplification clean up
Diego Novillo
dnovillo@google.com
Wed Dec 5 14:21:00 GMT 2007
Tom Tromey wrote:
> The first is diagnose_omp_structured_block_errors. I just turned this
> into an ordinary pass, which is run near the start of
> all_lowering_passes. I don't really understand why this couldn't be
> done this way on the trunk. But, since a lot of the test suite is not
> working on my branch, perhaps the answer is waiting to be found.
IIRC, it may have been a remnant of when we would expand OMP constructs
directly during gimplification, but I may be misremembering. As long as
we do this before final OMP expansion, this can be turned into a pass.
> The second is c_gimple_diagnostics_recursively. Here I took a
> potentially controversial route: I added a new langhook for a
> post-gimplification pass.
Oh noes! ;)
Seriously, it does not matter. OpenMP validation and initial processing
is highly front end specific, so none of these activities will ever be
done from language-independent layers. A langhook here is just fine.
Diego.
More information about the Gcc-patches
mailing list