Subzero: Use -filetype instead of -ias and -elf-writer.

Subzero: Use -filetype instead of -ias and -elf-writer.

This matches LLVM's -filetype={obj,asm} options, and also adds -filetype=iasm to get textual rendering with the integrated assembler.

It also removes the old illegal combination of -ias=0 -elf-writer.

BUG= none
R=jvoung@chromium.org, kschimpf@google.com

Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=d442e7eed84629ee885934e13a2fad5cf7909485

[Jim Stichnoth, 2015-02-12 20:44:27]

stichnot@chromium.org changed reviewers:
+ jvoung@chromium.org, kschimpf@google.com

[Karl, 2015-02-12 21:10:54]

Otherwise, LGTM.

https://codereview.chromium.org/920953002/diff/1/unittest/BitcodeMunge.cpp
File unittest/BitcodeMunge.cpp (left):

https://codereview.chromium.org/920953002/diff/1/unittest/BitcodeMunge.cpp#ol...
unittest/BitcodeMunge.cpp:31: Flags.setUseIntegratedAssembler(true); // for the
MINIMAL build
Should we be counting on the default for the assembler, or be safe in case it
changes again?

[jvoung (off chromium), 2015-02-12 21:20:40]

lgtm too

[Jim Stichnoth, 2015-02-12 21:45:15]

New patchsets have been uploaded after l-g-t-m from
kschimpf@google.com,jvoung@chromium.org

[Jim Stichnoth, 2015-02-12 21:45:50]

https://codereview.chromium.org/920953002/diff/1/unittest/BitcodeMunge.cpp
File unittest/BitcodeMunge.cpp (left):

https://codereview.chromium.org/920953002/diff/1/unittest/BitcodeMunge.cpp#ol...
unittest/BitcodeMunge.cpp:31: Flags.setUseIntegratedAssembler(true); // for the
MINIMAL build
On 2015/02/12 21:10:53, Karl wrote:
> Should we be counting on the default for the assembler, or be safe in case it
> changes again?

Good point, done.

[Jim Stichnoth, 2015-02-12 22:01:53]

Committed patchset #2 (id:20001) manually as
d442e7eed84629ee885934e13a2fad5cf7909485 (presubmit successful).