commit 66c37275ea3acb768d371c46c06a91d7052db8a8
author Gabe Black <gabeblack@google.com> Sat Jan 27 03:15:13 2018 -0800
committer Gabe Black <gabeblack@google.com> Sat Jan 27 20:28:40 2018 +0000
tree bfb0b893f0e78591030f10e1aafee83b290bf332
parent 3ccef3dd7702530718f145c4c30061688ebe276f

base: Get bitunions to compile on clang 3.8.

clang was getting very upset and interpretting a member function
pointer as a call to the actual underlying function, and then
complaining that it was a non-static function call without an instance.

It seems what it was really upset about was that the class who's scope
the member function pointer belonged to (the current class) wasn't done
being defined. This *should* be ok as far as I can tell, but clang was
having none of it.

This change reworks how the type of the setter function arguments are
determined to work around that limitation. The bitunion test was run
with clang++ and g++ and both pass, and I've built gem5.opt for ARM
successfully.

Change-Id: Ib9351784a897af4867fe08045577e0247334ea11
Reviewed-on: https://gem5-review.googlesource.com/7581
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Gabe Black <gabeblack@google.com>