Move arm-nacl DL description string setting from ctor to setABI.

Move arm-nacl DL description string setting from ctor to setABI.

ARMTargetInfo sets the description string from setABI and not the
ctor, so don't do that under the NaClTargetInfo<ARM...> ctor.

Otherwise, the NaClTargetInfo<ARM...> ctor will set the description
string, then CreateTargetInfo will call setABI() and replace the
description string with another one. Move the arm-nacl setting to
one place (setABI). Change the existing test to pass
"-target-abi aapcs-linux", like what the driver would normally do
for NaCl to test this.

Also fix up the target-data test to account for the f80 localmods.

BUG= https://code.google.com/p/nativeclient/issues/detail?id=3930
R=dschuff@chromium.org

Committed: ea86293b0aae2ab43e317256fe042225209de6a0

[jvoung (off chromium), 2014-11-11 17:25:45]

jvoung@chromium.org changed reviewers:
+ dschuff@chromium.org

[Derek Schuff, 2014-11-11 17:59:30]

lgtm

https://codereview.chromium.org/714943002/diff/20001/lib/Basic/Targets.cpp
File lib/Basic/Targets.cpp (right):

https://codereview.chromium.org/714943002/diff/20001/lib/Basic/Targets.cpp#ne...
lib/Basic/Targets.cpp:661: // Handled in ARM's setABI().
I guess if this is going to change upstream we should put a localmod marker here
with a comment that setABI is going away?

[jvoung (off chromium), 2014-11-11 18:23:49]

https://codereview.chromium.org/714943002/diff/20001/lib/Basic/Targets.cpp
File lib/Basic/Targets.cpp (right):

https://codereview.chromium.org/714943002/diff/20001/lib/Basic/Targets.cpp#ne...
lib/Basic/Targets.cpp:661: // Handled in ARM's setABI().
On 2014/11/11 17:59:30, Derek Schuff wrote:
> I guess if this is going to change upstream we should put a localmod marker
here
> with a comment that setABI is going away?

Done

[jvoung (off chromium), 2014-11-11 18:27:32]

Committed patchset #3 (id:40001) manually as
ea86293b0aae2ab43e317256fe042225209de6a0 (presubmit successful).