[llvm-commits] Name change in X86IselLowering - Please review

Nadav Rotem nrotem at apple.com
Tue Nov 27 09:25:37 PST 2012


Elena, 

This makes sense to me. 

Nadav

On Nov 27, 2012, at 2:41 AM, "Demikhovsky, Elena" <elena.demikhovsky at intel.com> wrote:

> Hi,
> 
> I'd like to change hasAVX() to hasFp256() and hasAVX2() to hasInt256() in X86IselLowering.cpp.
> I do not remove hasAVX2 and hasAVX, just duplicate them.
> Actually, the target that supports integer-256 also supports fp-256, but the suggested names fit good to the code.
> I need this change for future development.
> 
> - Elena
> 
> 
> ---------------------------------------------------------------------
> Intel Israel (74) Limited
> 
> This e-mail and any attachments may contain confidential material for
> the sole use of the intended recipient(s). Any review or distribution
> by others is strictly prohibited. If you are not the intended
> recipient, please contact the sender and delete all copies.
> <AVX.diff>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list