On 04/14/2014 05:25 PM, yijiang wrote: > + void setHasExtractBitsInsn(bool hasExtractInsn = false) { > + HasExtractBitsInsn = hasExtractInsn; > + } The default argument should be true, since that's what all the others do and since the default value is already false