Subzero: Fix -build-atts option.

Subzero: Fix -build-atts option.

If the default --target option is not actually included in the Subzero build (via llvm/Config/SZTargets.def), then "pnacl-sz --build-atts" will fail.  This is because the attribute printing is done after the GlobalContext is created, which does some amount of Target initialization.

The fix is to move the attribute printing to a point after the flags are parsed and the output streams are created, but before the GlobalContext is created.  This basically disables --build-atts in the browser build, but that should be OK.

BUG= none
R=kschimpf@google.com

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

[Jim Stichnoth, 2016-01-13 21:10:57]

Description was changed from

==========
Subzero: Fix -build-atts option.

BUG= none
==========

to

==========
Subzero: Fix -build-atts option.

If the default --target option is not actually included in the Subzero build
(via llvm/Config/SZTargets.def), then "pnacl-sz --build-atts" will fail.  This
is because the attribute printing is done after the GlobalContext is created,
which does some amount of Target initialization.

The fix is to move the attribute printing to a point after the flags are parsed
and the output streams are created, but before the GlobalContext is created. 
This basically disables --build-atts in the browser build, but that should be
OK.

BUG= none
==========

[Jim Stichnoth, 2016-01-13 21:10:57]

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

[Karl, 2016-01-13 21:14:01]

lgtm

[Jim Stichnoth, 2016-01-13 21:48:49]

Description was changed from

==========
Subzero: Fix -build-atts option.

If the default --target option is not actually included in the Subzero build
(via llvm/Config/SZTargets.def), then "pnacl-sz --build-atts" will fail.  This
is because the attribute printing is done after the GlobalContext is created,
which does some amount of Target initialization.

The fix is to move the attribute printing to a point after the flags are parsed
and the output streams are created, but before the GlobalContext is created. 
This basically disables --build-atts in the browser build, but that should be
OK.

BUG= none
==========

to

==========
Subzero: Fix -build-atts option.

If the default --target option is not actually included in the Subzero build
(via llvm/Config/SZTargets.def), then "pnacl-sz --build-atts" will fail.  This
is because the attribute printing is done after the GlobalContext is created,
which does some amount of Target initialization.

The fix is to move the attribute printing to a point after the flags are parsed
and the output streams are created, but before the GlobalContext is created. 
This basically disables --build-atts in the browser build, but that should be
OK.

BUG= none
R=kschimpf@google.com

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

[Jim Stichnoth, 2016-01-13 21:48:50]

Committed patchset #1 (id:1) manually as
686f35e569f7fac14d357550faed542de261415b (presubmit successful).