> But that looks like the correct thing to do. It's not. The patch I reverted changes it so that no matter what, void_list_node = build_tree_list (NULL_TREE, void_type_node);.
Before, each front-end set it in their own way, but they all set it via void_list_node = build_tree_list (NULL_TREE, void_type_node); or a synonym anyway. So while the patch made sense in a java-free context, given that java sets it a different way, I can't see a world in which this commit stays active and Java works, unless we find a way to set it in tree.cc for every language except Java.