S390: Initial impl of regexp

S390: Initial impl of regexp

R=danno@chromium.org,jkummerow@chromium.org,jochen@chromium.org,joransiu@ca.ibm.com,michael_dawson@ca.ibm.com,mbrandy@us.ibm.com
BUG=

Committed: https://crrev.com/be33a9d89da370aee73a8b31011ab5b1b46dd0d7
Cr-Commit-Position: refs/heads/master@{#34592}

[john.yan, 2016-03-07 21:05:31]

The CQ bit was checked by jyan@ca.ibm.com to run a CQ dry run

[commit-bot: I haz the power, 2016-03-07 21:05:38]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1768383002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1768383002/1

[john.yan, 2016-03-07 21:14:20]

S390: Initial impl of regexp

[commit-bot: I haz the power, 2016-03-07 21:26:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-07 21:26:08]

Dry run: This issue passed the CQ dry run.

[Jakob Kummerow, 2016-03-08 09:50:27]

LGTM (rubberstamping *-s390.{h,cc})

Jochen, please give src/regexp/OWNERS approval.

https://codereview.chromium.org/1768383002/diff/1/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/1768383002/diff/1/BUILD.gn#newcode1680
BUILD.gn:1680: "src/regexp/s390/regexp-macro-assembler-s390.cc",
nit: if you care about maintaining alphabetic ordering, "r" comes before "s" ;-)

https://codereview.chromium.org/1768383002/diff/1/tools/gyp/v8.gyp
File tools/gyp/v8.gyp (right):

https://codereview.chromium.org/1768383002/diff/1/tools/gyp/v8.gyp#newcode1600
tools/gyp/v8.gyp:1600: '../../src/regexp/s390/regexp-macro-assembler-s390.cc',
same nit here

[john.yan, 2016-03-08 13:01:08]

Patchset #2 (id:20001) has been deleted

[john.yan, 2016-03-08 13:09:42]

On 2016/03/08 09:50:27, Jakob wrote:
> LGTM (rubberstamping *-s390.{h,cc})
> 
> Jochen, please give src/regexp/OWNERS approval.
> 
> https://codereview.chromium.org/1768383002/diff/1/BUILD.gn
> File BUILD.gn (right):
> 
> https://codereview.chromium.org/1768383002/diff/1/BUILD.gn#newcode1680
> BUILD.gn:1680: "src/regexp/s390/regexp-macro-assembler-s390.cc",
> nit: if you care about maintaining alphabetic ordering, "r" comes before "s"
;-)
> 
> https://codereview.chromium.org/1768383002/diff/1/tools/gyp/v8.gyp
> File tools/gyp/v8.gyp (right):
> 
> https://codereview.chromium.org/1768383002/diff/1/tools/gyp/v8.gyp#newcode1600
> tools/gyp/v8.gyp:1600: '../../src/regexp/s390/regexp-macro-assembler-s390.cc',
> same nit here

thanks! file names are now sorted in alphabetic ordering by "sort"

[Yang, 2016-03-08 14:35:37]

On 2016/03/08 13:09:42, john.yan wrote:
> On 2016/03/08 09:50:27, Jakob wrote:
> > LGTM (rubberstamping *-s390.{h,cc})
> > 
> > Jochen, please give src/regexp/OWNERS approval.
> > 
> > https://codereview.chromium.org/1768383002/diff/1/BUILD.gn
> > File BUILD.gn (right):
> > 
> > https://codereview.chromium.org/1768383002/diff/1/BUILD.gn#newcode1680
> > BUILD.gn:1680: "src/regexp/s390/regexp-macro-assembler-s390.cc",
> > nit: if you care about maintaining alphabetic ordering, "r" comes before "s"
> ;-)
> > 
> > https://codereview.chromium.org/1768383002/diff/1/tools/gyp/v8.gyp
> > File tools/gyp/v8.gyp (right):
> > 
> >
https://codereview.chromium.org/1768383002/diff/1/tools/gyp/v8.gyp#newcode1600
> > tools/gyp/v8.gyp:1600:
'../../src/regexp/s390/regexp-macro-assembler-s390.cc',
> > same nit here
> 
> thanks! file names are now sorted in alphabetic ordering by "sort"

LGTM

[john.yan, 2016-03-08 14:47:35]

The CQ bit was checked by jyan@ca.ibm.com

[john.yan, 2016-03-08 14:47:36]

The patchset sent to the CQ was uploaded after l-g-t-m from
jkummerow@chromium.org
Link to the patchset: https://codereview.chromium.org/1768383002/#ps40001
(title: "rebase to master and sort file in alphabetic ordering")

[commit-bot: I haz the power, 2016-03-08 14:47:42]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1768383002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1768383002/40001

[commit-bot: I haz the power, 2016-03-08 15:08:06]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-03-08 15:08:26]

Description was changed from

==========
S390: Initial impl of regexp

R=danno@chromium.org,jkummerow@chromium.org,jochen@chromium.org,joransiu@ca.i...
BUG=
==========

to

==========
S390: Initial impl of regexp

R=danno@chromium.org,jkummerow@chromium.org,jochen@chromium.org,joransiu@ca.i...
BUG=

Committed: https://crrev.com/be33a9d89da370aee73a8b31011ab5b1b46dd0d7
Cr-Commit-Position: refs/heads/master@{#34592}
==========

[commit-bot: I haz the power, 2016-03-08 15:08:27]

Patchset 2 (id:??) landed as
https://crrev.com/be33a9d89da370aee73a8b31011ab5b1b46dd0d7
Cr-Commit-Position: refs/heads/master@{#34592}