[llvm-commits] [llvm] r90108 - in /llvm/trunk: include/llvm/CodeGen/SelectionDAG.h lib/CodeGen/SelectionDAG/LegalizeTypes.cpp lib/CodeGen/SelectionDAG/LegalizeTypes.h lib/CodeGen/SelectionDAG/LegalizeVectorOps.cpp lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp lib/CodeGen/SelectionDAG/SelectionDAG.cpp lib/Target/X86/X86ISelLowering.cpp

[Duncan Sands]

Hi Mon Ping,

> The solution for divide by unrolling is X86 specific.  Widening the divisor by 1 would always safe but could lead to an extra scalar divide being generated for this case.

yes, that might be so.  However, the extra values generated by widening usually
won't actually be used (at least I don't think so) so I would expect any such
extra divisions to be discarded by the DAG combiner.

Ciao,

Duncan.