On Thu, 5 Jun 2014, Ralf Baechle wrote: > > Ralf: As a side note, while looking at > > arch/mips/include/asm/div64.h, I saw that the implementation of > > __div64_32 in that file will be unused, and is also completely > > broken due to the first parameter never being used. > > Seems I broke c21004cd5b4cb7d479514d470a62366e8307412c "MIPS: Rewrite > <asm/div64.h> to work with gcc 4.4.0." Took only five years until > somebody noticed ... Well, it's not really possible to track all the breakage introduced unless at least a warning is spat for some config. Which is why I think it's a good idea to have all non-trivial changes put through a review process or at least posted to the relevant mailing list. This change didn't appear anywhere, only the commit log message was posted, so there was no chance to spot the damage, unless there was someone who actively studied changes made to the tree. Maciej
