[llvm-commits] [PATCH] revise/enhance atomic primitive code generation

Chandler Carruth chandlerc at google.com
Wed Jul 25 15:34:32 PDT 2012


First off, these sound like fantastic enhancements! =] Thanks for pulling
them out.

Can you split this into separate patches and post them individually for
review? I think each of your bullet points would make for a good patch to
review.

On Wed, Jul 25, 2012 at 3:06 PM, Michael Liao <michael.liao at intel.com>wrote:

> - unify the logic in SelectAtomicLoadAdd and SelectAtomicLoadArith and
>   merge them together
> - refine spin-loop to reduce one unnecessary load
> - add missing i8 max/min/umax/umin
> - add missing i64 max/min/umax/umin on 32-bit target
> - refine atomic instruction td files to use the template for groups of
>   instructions
> - Output 'lock' prefix in assembler printer to simplify the assembly
>   text in td files
>
> Please review the attached patch and commit if it's OK.
>
> Yours
> - Michael
>
> ---
> [1] http://software.intel.com/file/36945
> [2]
>
> http://software.intel.com/en-us/blogs/2012/02/07/transactional-synchronization-in-haswell/
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120725/559c17a0/attachment.html>


More information about the llvm-commits mailing list