Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(919)

Issue 301023004: MIPS: Refactor HType to get rid of various hacks. (Closed)

Created:
6 years, 6 months ago by kilvadyb
Modified:
6 years, 6 months ago
CC:
v8-dev
Base URL:
https://github.com/v8/v8.git@gbl
Visibility:
Public.

Description

MIPS: Refactor HType to get rid of various hacks. Port r21578 (dcf13aa) Original commit message: - Move HType to it's own file. - Add HType::HeapObject and some other useful types. - Get rid of the broken and useless HType::NonPrimitive. - Introduce HType::FromType() to convert from HeapType to HType. - Also add unit tests for HType. - Fix types in Crankshaft. BUG=

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -7 lines) Patch
M src/mips/lithium-codegen-mips.cc View 6 chunks +6 lines, -6 lines 0 comments Download
M src/mips/lithium-mips.cc View 1 chunk +3 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
kilvadyb
6 years, 6 months ago (2014-05-29 09:16:14 UTC) #1
Benedikt Meurer
LGTM
6 years, 6 months ago (2014-05-29 09:16:40 UTC) #2
Paul Lind
6 years, 6 months ago (2014-05-29 13:43:20 UTC) #3
Thanks for the review, Benedikt.
Committed as r21580.

Powered by Google App Engine
This is Rietveld 408576698