Re: [PATCH 48/50] varasm.c:const_rtx_hash

2014-08-05 Thread Jeff Law
On 08/03/14 08:42, Richard Sandiford wrote: const_rtx_hash_1 had code to hash all elements of a CONST_VECTOR, but const_rtx_hash also hashes all subrtxes, so we'd end up hashing the same thing twice. This looked unintentional so I just removed the CONST_VECTOR case. If instead it was a delibera

[PATCH 48/50] varasm.c:const_rtx_hash

2014-08-03 Thread Richard Sandiford
const_rtx_hash_1 had code to hash all elements of a CONST_VECTOR, but const_rtx_hash also hashes all subrtxes, so we'd end up hashing the same thing twice. This looked unintentional so I just removed the CONST_VECTOR case. If instead it was a deliberate decision then I think it deserves a comment