[Diffusion] rL358949: [PowerPC] [Clang] Port MMX intrinsics and basic test cases to Power

Hubert Tong via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 4 15:23:56 PDT 2019


hubert.reinterpretcast added inline comments.

/cfe/trunk/lib/Driver/ToolChains/PPCLinux.cpp:26 I'm not particularly concerned about the scoping of the work; however, the `ppc_wrappers` directory is not at all indicative of that scoping. It neither indicates the scope in terms of purpose (providing x86 vector intrinsics) nor does it indicate the scope in terms of applicability (64-bit Linux). I would suggest that the header belongs as part of a deeper structure that can be used for other purposes as well `ppc_wrappers/ppc64common/linux/`. If the header becomes applicable to more platforms, we can move it with the possibility of additionally placing headers that use `#include_next` as appropriate.

https://reviews.llvm.org/rL358949





More information about the cfe-commits mailing list