This is an archive of the discontinued LLVM Phabricator instance.

[PowerPC] Add implementation for -msave-toc-indirect option - clang portion
Needs ReviewPublic

Authored by syzaara on Oct 27 2017, 8:42 AM.

Details

Summary

Add clang option -msave-toc-indirect for PowerPC.

Diff Detail

Event Timeline

syzaara created this revision.Oct 27 2017, 8:42 AM
hfinkel edited edge metadata.Oct 27 2017, 11:39 AM

Please upload diffs with full context.

include/clang/Driver/Options.td
1907

You also need mno_save_toc_indirect here too.

jtony added inline comments.Nov 28 2017, 11:48 AM
test/Driver/ppc-features.cpp
140

We probably need to add test for mno_save_toc_indirect also once you add it according to Hal's suggestion.

sfertile added inline comments.Nov 29 2017, 9:49 AM
include/clang/Driver/Options.td
1907

nit: The new option splits the mvsx and mno_vsx pair. I think we should keep the related options together.