[PATCH] D79194: Add header guards for header files that should not be included on the PS4 platform

Douglas Yung via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 30 12:21:12 PDT 2020


dyung created this revision.
dyung added a reviewer: craig.topper.
dyung added a project: clang.

For the PS4 platform, we want to exclude certain headers from being included because we do not support the features they provide. To achieve this, we do not include any header files by default, and rely on the cpu target features to selectively enable supported header files. Compilers targeting the PS4 define __SCE__ and individual features will be set automatically by the compiler's -target-cpu option (btver2 for the PS4).

The test change is needed because with this change, the test was failing because our __SCE__ define was excluding everything, and there is no corresponding -target-cpu specified by the test to turn individual features back on. When no -target-cpu is selected, a generic cpu is assumed by the compiler with no additional feature support enabled. The fix I am adding is explicitly running each command with both 32/64bit generic triples.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D79194

Files:
  clang/lib/Headers/bmiintrin.h
  clang/lib/Headers/immintrin.h
  clang/lib/Headers/x86intrin.h
  clang/test/Headers/x86intrin-2.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D79194.261302.patch
Type: text/x-patch
Size: 21004 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200430/635a5f8f/attachment-0001.bin>


More information about the cfe-commits mailing list