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

Tom Stellard tom at stellard.net
Tue Mar 10 07:37:03 PDT 2015


On Tue, Mar 10, 2015 at 01:28:59PM +0100, Joerg Sonnenberger wrote:
> Tom, what about you?

These LGTM, go ahead and merge.

-Tom

> 
> Joerg
> 
> On Tue, Mar 10, 2015 at 09:44:11AM +0000, Daniel Sanders wrote:
> > 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
> > 
> > 
> > _______________________________________________
> > 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