[PATCH] D36555: Move x86-specific sources to x86-specific source lists.
Sterling Augustine via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 22 15:23:49 PDT 2017
saugustine added a reviewer: weimingz.
saugustine added inline comments.
================
Comment at: compiler-rt/lib/builtins/CMakeLists.txt:223
+ cpu_model.c
+ divxc3.c
+ fixxfdi.c
----------------
mgorny wrote:
> This and the following files have only:
>
> ```
> #if !_ARCH_PPC
> ```
>
> so I suppose it's currently included on more targets than x86. If it's really not useful there, I think it'd be better to update the `#if`s first (and preferably get the author to review).
As discussed in D36764, these all assume 80-bit floats, and therefore are only useful on x86 (and ancient 68ks)--and, in fact, the tests for these functions are disabled for everything but x86. I have left these #ifs in for now, but will remove them after this patch gets submitted (and add comments about them needing 80-bit floats). Otherwise, powerpc builds will be broken.
https://reviews.llvm.org/D36555
More information about the cfe-commits
mailing list