This is an archive of the discontinued LLVM Phabricator instance.

Complete setting the baseline for ppc64le
ClosedPublic

Authored by nemanjai on Jan 28 2015, 2:32 PM.

Details

Reviewers
wschmidt
hfinkel
Summary

As was uncovered by the failing test case (when run on non-PPC platforms), the feature set when compiling with -march=ppc64le was not being picked up. This change ensures that if the -mcpu option is not specified, the correct feature set is picked up regardless of whether we are on PPC or not.

Diff Detail

Repository
rL LLVM

Event Timeline

nemanjai updated this revision to Diff 18921.Jan 28 2015, 2:32 PM
nemanjai retitled this revision from to Complete setting the baseline for ppc64le.
nemanjai updated this object.
nemanjai edited the test plan for this revision. (Show Details)
nemanjai added reviewers: hfinkel, wschmidt.
nemanjai set the repository for this revision to rL LLVM.
nemanjai added a subscriber: Unknown Object (MLST).
hfinkel accepted this revision.Jan 28 2015, 3:17 PM
hfinkel edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Jan 28 2015, 3:17 PM
wschmidt edited edge metadata.Jan 29 2015, 5:23 AM

Thanks, Hal. I will commit on Nemanja's behalf today.

wschmidt closed this revision.Jan 29 2015, 8:01 AM

Committed as r227455. Thanks, Nemanja!