This patch adds missing pieces to clang, including the PS4 toolchain
definition, added warnings, PS4 defaults, and Driver changes needed for
our compiler.
Patch extracted by me, Katya Romanova, and Pierre Gousseau.
Paths
| Differential D11279
Initial patch for PS4 toolchain ClosedPublic Authored by filcab on Jul 16 2015, 2:59 PM.
Details Summary This patch adds missing pieces to clang, including the PS4 toolchain Patch extracted by me, Katya Romanova, and Pierre Gousseau.
Diff Detail Event Timelinefilcab updated this object. Comment Actions LGTM. However, since we were preparing a patch together with Filipe and Pierre, I'm as bias as they are :) We addressed all my comments before Filipe submitted this patch. Kattya. filcab edited edge metadata. Comment ActionsUpdate for clang API change. Comment Actions Added a bunch of inline comments. The easiest way for this is probably going to be to split this out into a few separate patches where we get the skeleton in and then start filling it out. Thanks! -eric
This revision is now accepted and ready to land.Jul 29 2015, 6:03 PM Comment Actions Thank you, Eric. I would prefer to not split this into "add the toolchain, then fill it out more" (if possible), since we already have this in our internal branch. Having two bigger merges (especially if one is "non-working") or waiting until we have everything to do the merge wouldn't be ideal.
filcab added inline comments.
Comment Actions Replying to the inline comments. I know we're waiting on the linker stuff, but Katya made me realize I hadn't replied to your comments. -eric
filcab marked 12 inline comments as done. filcab edited edge metadata. Comment ActionsAddressed review comments. This revision is now accepted and ready to land.Sep 10 2015, 6:47 PM echristo edited edge metadata. Comment ActionsOne set of inline comments, and also: test/Driver/Inputs/scei-ps4_tree/target/include_common/.keep eh? -eric
Comment Actions test/Driver/Inputs/scei-ps4_tree/target/include_common/ and test/Driver/Inputs/scei-ps4_tree/target/include/ are expected to be present by a test ps4-header-search.c. This test checks that these directories are found in the header search path. If these directories are not found, the compiler complains and the test fails.
Comment Actions
How about just a README file in the directories saying this rather than the .keep file? :) -eric
Comment Actions Hi Eric, Comment Actions
Huh. I guess not. (Well, I mean yes, but if I want that done I can do it myself and keep consistency in this commit). -eric Closed by commit rL248546: This patch adds missing pieces to clang, including the PS4 toolchain (authored by kromanova). · Explain WhySep 24 2015, 3:33 PM This revision was automatically updated to reflect the committed changes. Comment Actions Would this cause bunch of failures if target is *-ps4 and SDK_DIR is empty? clang-3.8: error: unable to find PS4 system headers directory, expected to be in '/home/buildbot/Buildbot/Slave/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fast/llvm.obj/bin/../../target/include'
Revision Contents
Diff 30062 include/clang/Basic/DiagnosticDriverKinds.td
include/clang/Basic/DiagnosticGroups.td
include/clang/Driver/Options.td
lib/Driver/Driver.cpp
lib/Driver/ToolChains.h
lib/Driver/ToolChains.cpp
lib/Driver/Tools.h
lib/Driver/Tools.cpp
lib/Frontend/InitHeaderSearch.cpp
test/Driver/Inputs/scei-ps4_tree/target/include_common/.keep
test/Driver/debug-options.c
test/Driver/ps4-header-search.c
test/Driver/ps4-linker-non-win.c
test/Driver/ps4-linker-win.c
test/Driver/ps4-pic.c
test/Driver/ps4-sdk-root.c
test/Driver/rtti-options.cpp
test/Driver/stack-protector.c
|
Are the existing header warnings insufficient?