RFR: 8075247: Cleanup specialized_oop_closures.hpp
thomas.schatzl at oracle.com
Mon Mar 16 20:45:08 UTC 2015
On Mon, 2015-03-16 at 17:04 +0100, Stefan Karlsson wrote:
> Please review this patch to rename and simplify some macros in
> specialized_oop_closures.hpp and g1_specialized_oop_closures.hpp.
> The patch contains the following changes:
> 1) The FURTHER_SPECIALIZED* macros are only used by G1, so they have
> been renamed into SPECIALIZED_*G1.
> 2) Removed some unnecessary #ifdefs. The compile will catch the error if
> we try to redefine the macro.
> 3) Gather all CMS specific macros inside a new
> SPECIALIZED_OOP_OOP_ITERATE_CLOSURES_CMS macro.
I am not completely sure why in many files touched in this change both
cpp and the corresponding hpp files include
specialized_oop_closures.hpp. Is there a particular reason to do so?
More information about the hotspot-gc-dev