[llvm-commits] [llvm] r131012 - in /llvm/trunk: include/llvm/Target/ lib/CodeGen/ lib/CodeGen/SelectionDAG/ lib/Target/ARM/ lib/Target/Alpha/ lib/Target/Blackfin/ lib/Target/CellSPU/ lib/Target/MBlaze/ lib/Target/MSP430/ lib/Target/Mips/ lib/Target/PTX/ lib/Target/PowerPC/ lib/Target/Sparc/ lib/Target/SystemZ/ lib/Target/X86/ lib/Target/XCore/

[Duncan Sands]

Hi Eli,

> +  /// setPrefFunctionAlignment - Set the target's preferred function alignment.
> +  /// This should be set if there is a small performance benefit to
> +  /// higher-than-minimum alignment

according to this comment you should not set this if there is a large
performance benefit to doing so :)  I reckon it would be better to drop
the word "small",

> +  /// PrefFunctionAlignment - The perferred function alignment (used when

perferred -> preferred

Ciao, Duncan.