3.6.1 patch nomination: r227087 and r227089 ("[mips] Enable arithmetic and binary operations for the i128 data type")

Daniel Sanders Daniel.Sanders at imgtec.com
Tue Mar 10 02:44:11 PDT 2015


LGTM.

> -----Original Message-----
> From: Dimitry Andric [mailto:dimitry at andric.com]
> Sent: 09 March 2015 20:56
> To: Daniel Sanders
> Cc: Tom Stellard; llvm-commits; Sean Bruno
> Subject: 3.6.1 patch nomination: r227087 and r227089 ("[mips] Enable
> arithmetic and binary operations for the i128 data type")
> 
> Hi Daniel,
> 
> Can you please approve these two commits for merging into the release_36
> branch, targeting 3.6.1?
> 
> ==========================================================
> ============
> Author: vkalintiris
> Date: Mon Jan 26 06:04:40 2015
> New Revision: 227087
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=227087&view=rev
> Log:
> [mips] Add tests for bitwise binary and integer arithmetic operators.
> 
> Reviewers: dsanders
> 
> Subscribers: llvm-commits
> 
> Differential Revision: http://reviews.llvm.org/D7125
> ==========================================================
> ============
> 
> and
> 
> ==========================================================
> ============
> Author: vkalintiris
> Date: Mon Jan 26 06:33:22 2015
> New Revision: 227089
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=227089&view=rev
> Log:
> [mips] Enable arithmetic and binary operations for the i128 data type.
> 
> Summary:
> This patch adds support for some operations that were missing from
> 128-bit integer types (add/sub/mul/sdiv/udiv... etc.). With these
> changes we can support the __int128_t and __uint128_t data types
> from C/C++.
> 
> Depends on D7125
> 
> Reviewers: dsanders
> 
> Subscribers: llvm-commits
> 
> Differential Revision: http://reviews.llvm.org/D7143
> ==========================================================
> ============
> 
> -Dimitry





More information about the llvm-commits mailing list