[compiler-rt] r231965 - Refactor float to integer conversion to share the same code.

Joerg Sonnenberger joerg at britannica.bec.de
Thu Mar 12 14:35:02 PDT 2015


On Thu, Mar 12, 2015 at 12:09:56PM -0700, Justin Bogner wrote:
> Joerg Sonnenberger <joerg at bec.de> writes:
> > Author: joerg
> > Date: Wed Mar 11 16:13:56 2015
> > New Revision: 231965
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=231965&view=rev
> > Log:
> > Refactor float to integer conversion to share the same code.
> > 80bit Intel/PPC long double is excluded due to lacking support
> > for the abstraction. Consistently provide saturation logic.
> > Extend to long double on 128bit IEEE extended platforms.
> 
> I'm getting warnings after this change. Apparently the C grammar doesn't
> like empty TUs:
> 
> ../llvm/projects/compiler-rt/lib/builtins/fixunssfti.c:23:7: error: ISO C requires a translation unit to contain at least one declaration [-Werror,-Wempty-translation-unit]

Which compiler with which flags? Note that we use the same logic e.g. in
addtf3.c too.

Joerg



More information about the llvm-commits mailing list