I've messed around with it locally and nothing seemed to stop working.
Would there be any interest in adding a newer clang configuration with
-mno-sse removed?
On Thu, Apr 4, 2019 at 1:12 PM Andrew Fish wrote:
>
>
> On Apr 4, 2019, at 1:09 PM, Gary Zibrat via Groups.Io <
>
> On Apr 4, 2019, at 1:09 PM, Gary Zibrat via Groups.Io
> wrote:
>
> It seems like the "-mno-mmx -msoft-float -mno-implicit-float" part of the
> flags should be enough to handle that now.
> Is it ok to take -mno-sse out (in my local branch) for new versions of clang
> without having to
It seems like the "-mno-mmx -msoft-float -mno-implicit-float" part of the
flags should be enough to handle that now.
Is it ok to take -mno-sse out (in my local branch) for new versions of
clang without having to worry about silent failures?
Since EFIAPI uses the windows ABI, it should be ok to use
Gary,
The edk2 is freestanding UEFI firmware and the runtime does not support
floating point exceptions.
I think a long time ago there was an optimization bug and the -mno-sse fixed
it.
The UEFI x64 ABI is Windows like and not Sys V like.
Thanks,
Andrew Fish
> On Apr 4, 2019, at 10:13
Does anyone know why -mno-sse is specified on Clang x64?
https://github.com/tianocore/edk2/commit/6f756db5ea057ff10dd168c7627163cec046a3d9
There doesn't seem to be a corresponding flag for other compilers.
I ask because a commit in LLVM breaks compilation of EDK2: