[committed,v2,11] ranger: Fix `-Werror' build error with `ranger_cache::push_poor_value'
Commit Message
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
===================================================================
@@ -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.