r207829 - CodeGen: rename __builtin_arm_sevl to __sevl

Yi Kong kongy.dev at gmail.com
Mon Jun 30 08:41:33 PDT 2014


Saleem,

On 30 June 2014 16:19, Renato Golin <renato.golin at linaro.org> wrote:
> On 30 June 2014 16:14, Yi Kong <kongy.dev at gmail.com> wrote:
>> I don't quite understand what you were asking me to do. Do you mean
>> moving MSVC specific tests into a separate directory?
>
> So, if I got it right, MSVC *needs* the builtins to have the short
> name, because it can't be on headers (like ACLE), but we only have one
> namespace for builtins (ARM) in the table-gen file.
>
> My question is, and I'm only guessing here, if we could separate the
> MSVC definitions in the table-gen file to only generate the short
> versions for Windows and the long versions for everything else. I
> don't think it's that easy. Another option would be to have both, but
> that would conflict with the headers, which won't work anywhere.
>
> I'm not sure what the solution is, and to be honest, I don't know the
> Windows environment that well to have a strong opinion. The best
> person to solve this issue would be Saleem, as he probably has a
> better idea on what changes would be necessary to keep both short and
> long versions as well as the header.
>
> cheers,
> --renato



More information about the cfe-commits mailing list