silence qcms __force_align_arg_pointer__ warning on mips too

silence qcms __force_align_arg_pointer__ warning on mips too

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=209815

[Mostyn Bramley-Moore, 2013-06-27 22:51:08]

This small patch is an extension of this previous CL:
https://chromiumcodereview.appspot.com/11365242

[Sam Clegg, 2013-06-27 23:22:04]

lgtm.

That list is getting long now, might be better to invert it?

[Nico, 2013-06-27 23:23:00]

lgtm

[Mostyn Bramley-Moore, 2013-06-28 05:49:43]

On 2013/06/27 23:22:04, Sam Clegg wrote:
> That list is getting long now, might be better to invert it?

Is this sufficient?
#elif defined(__GNUC__) && defined(__i386__)

[Sam Clegg, 2013-06-28 14:56:03]

On 2013/06/28 05:49:43, Mostyn Bramley-Moore wrote:
> On 2013/06/27 23:22:04, Sam Clegg wrote:
> > That list is getting long now, might be better to invert it?
> 
> Is this sufficient?
> #elif defined(__GNUC__) && defined(__i386__)

I wouldn't know without further investigation I'm afraid.  Looks right to me but
would need to verify.

[Sam Clegg, 2013-07-01 23:01:36]

On 2013/06/28 14:56:03, Sam Clegg wrote:
> On 2013/06/28 05:49:43, Mostyn Bramley-Moore wrote:
> > On 2013/06/27 23:22:04, Sam Clegg wrote:
> > > That list is getting long now, might be better to invert it?
> > 
> > Is this sufficient?
> > #elif defined(__GNUC__) && defined(__i386__)
> 
> I wouldn't know without further investigation I'm afraid.  Looks right to me
but
> would need to verify.

Feel free to ignore my nit and land this as is if you prefer.

[commit-bot: I haz the power, 2013-07-02 07:03:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mostynb@opera.com/18112005/1

[commit-bot: I haz the power, 2013-07-02 07:17:12]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[Mostyn Bramley-Moore, 2013-07-02 07:29:39]

Hmm, no OWNERS for third_party/qcms ...

@cpu: can you give this a thumbs-up?

[cpu_(ooo_6.6-7.5), 2013-07-02 20:04:40]

lgtm.

In the future you don't need root owners approval for changes in third party,
the root owners are only needed when adding a new third party package.

btw, this directory should have an OWNERS file.

[commit-bot: I haz the power, 2013-07-02 20:21:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mostynb@opera.com/18112005/1

[commit-bot: I haz the power, 2013-07-02 23:34:06]

Change committed as 209815