| Message ID | alpine.DEB.2.20.2111032131390.8821@tpp.orcam.me.uk |
|---|---|
| State | New |
| Headers | show |
| Series | [committed,v2,11] ranger: Fix `-Werror' build error with `ranger_cache::push_poor_value' | expand |
Index: gcc/gcc/gimple-range-cache.cc =================================================================== --- gcc.orig/gcc/gimple-range-cache.cc +++ gcc/gcc/gimple-range-cache.cc @@ -847,7 +847,7 @@ ranger_cache::register_dependency (tree // the request is actually made (ie, isn't a duplicate). bool -ranger_cache::push_poor_value (basic_block bb, tree name) +ranger_cache::push_poor_value (basic_block, tree) { // Disable poor value processing for GCC 11. It has been disabled in GCC 12 // as adding too much churn/compile time for too little benefit.
Remove a commit 86534c07a390 ("Disable poor value processing in ranger cache.") regression that caused GCC not to build anymore if `-Werror' has been enabled: .../gcc/gimple-range-cache.cc: In member function 'bool ranger_cache::push_poor_value(basic_block, tree)': .../gcc/gimple-range-cache.cc:850:44: error: unused parameter 'bb' [-Werror=unused-parameter] 850 | ranger_cache::push_poor_value (basic_block bb, tree name) | ~~~~~~~~~~~~^~ .../gcc/gimple-range-cache.cc:850:53: error: unused parameter 'name' [-Werror=unused-parameter] 850 | ranger_cache::push_poor_value (basic_block bb, tree name) | ~~~~~^~~~ To keep the change to the minimum mark the parameters reported unused. gcc/ * gimple-range-cache.cc (ranger_cache::push_poor_value): Mark parameters unused. --- On Wed, 3 Nov 2021, Jeff Law wrote: > If possible, just drop the argument's name. That's the C++ way to deal with > an unused parameter. Consider that pre-approved. If that doesn't work, then > adding the ATTRIBUTE_UNUSED is then approved. Umm, my 1990s dated proper C++ experience is surely rusty, disused and likely outdated. Thanks for the hint. Applied as shown then. Thanks for your review. Maciej --- gcc/gimple-range-cache.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) gcc-ranger-cache-push-poor-value-unused.diff