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

Issue 1799263002: Fix a warning about inline asm source/destination mismatches for cache_type_register_. (Closed)

Created:
4 years, 9 months ago by echristo.google
Modified:
4 years, 9 months ago
CC:
v8-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/v8/v8.git@master
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

Fix a warning about inline asm source/destination mismatches for cache_type_register_. The warning notes that we'd want a 'w' register here because the size of the operand is 32-bit, however, the instruction only takes an 'x' register and so force that using the 'x' modifier on the instruction. BUG= Committed: https://crrev.com/8d7399f9f8688a93137ced9e084bccb8cc1b075c Cr-Commit-Position: refs/heads/master@{#34766}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M src/base/cpu.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 9 (3 generated)
echristo.google
Hi Jochen, Please take a look at this quick warning fix. Thanks! -eric
4 years, 9 months ago (2016-03-14 22:18:25 UTC) #2
jochen (gone - plz use gerrit)
lgtm
4 years, 9 months ago (2016-03-15 05:04:15 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1799263002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1799263002/1
4 years, 9 months ago (2016-03-15 05:34:46 UTC) #5
echristo.google
Thanks!
4 years, 9 months ago (2016-03-15 05:36:43 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 9 months ago (2016-03-15 06:18:11 UTC) #7
commit-bot: I haz the power
4 years, 9 months ago (2016-03-15 06:19:00 UTC) #9
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/8d7399f9f8688a93137ced9e084bccb8cc1b075c
Cr-Commit-Position: refs/heads/master@{#34766}

Powered by Google App Engine
This is Rietveld 408576698