Hi Jeff,
The point of the patch is that the cmpstrnsi machine pattern should not be used to implement the memcmp builtin function. This is because a string comparison will terminate if two zero bytes are read whereas a memory comparison should continue.
OK. Kindof surprised this wasn't dealt with before.
Me too. Especially since the x86 port has a cmpstrnsi pattern and no cmpmemsi pattern...
I have applied the patch to the mainline and 4.6 and 4.5 branches. Cheers Nick