[NaCl SDK] Remove NACL_ARCH from commom makefiles.

[NaCl SDK] Remove NACL_ARCH from commom makefiles.

This macro was only used by one example, but was appearing
on the compile line for all examples.

Also, it can be detected more empirically at compile time
using GCC's builtins.

R=binji@chromium.org
BUG=None

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

[Sam Clegg, 2013-05-30 17:40:19]

Minor cleanup from my shuttle journey this morning...

https://codereview.chromium.org/15974011/diff/2001/native_client_sdk/src/tool...
File native_client_sdk/src/tools/host_vc.mk (right):

https://codereview.chromium.org/15974011/diff/2001/native_client_sdk/src/tool...
native_client_sdk/src/tools/host_vc.mk:30: ifeq ($(CONFIG),Release)
This is now consistent with the corresponding check in nacl_gcc.mk

[binji, 2013-05-30 17:53:30]

lgtm

https://codereview.chromium.org/15974011/diff/2001/native_client_sdk/src/tool...
File native_client_sdk/src/tools/nacl_gcc.mk (left):

https://codereview.chromium.org/15974011/diff/2001/native_client_sdk/src/tool...
native_client_sdk/src/tools/nacl_gcc.mk:49: # Architecture-specific flags
OK, but you should either remove their use below, or add in empty definitions
here, I think.

[Sam Clegg, 2013-05-30 17:57:06]

https://codereview.chromium.org/15974011/diff/2001/native_client_sdk/src/tool...
File native_client_sdk/src/tools/nacl_gcc.mk (left):

https://codereview.chromium.org/15974011/diff/2001/native_client_sdk/src/tool...
native_client_sdk/src/tools/nacl_gcc.mk:49: # Architecture-specific flags
On 2013/05/30 17:53:30, binji wrote:
> OK, but you should either remove their use below, or add in empty definitions
> here, I think.

I think it might be useful to have ARCH specific flags.

Just omitting them here is the same as having empty defs.  I'd rather omit them
in the name of keeping this file clean(er).

[binji, 2013-05-30 19:54:30]

https://codereview.chromium.org/15974011/diff/2001/native_client_sdk/src/tool...
File native_client_sdk/src/tools/nacl_gcc.mk (left):

https://codereview.chromium.org/15974011/diff/2001/native_client_sdk/src/tool...
native_client_sdk/src/tools/nacl_gcc.mk:49: # Architecture-specific flags
On 2013/05/30 17:57:06, Sam Clegg wrote:
> On 2013/05/30 17:53:30, binji wrote:
> > OK, but you should either remove their use below, or add in empty
definitions
> > here, I think.
> 
> I think it might be useful to have ARCH specific flags.
> 
> Just omitting them here is the same as having empty defs.  I'd rather omit
them
> in the name of keeping this file clean(er).

Yes, but now it's hard to tell that they even exist -- you'd have to look in the
nasty code below to see them, and what the names are.

[Sam Clegg, 2013-06-06 20:36:26]

Committed patchset #3 manually as r204582 (presubmit successful).