[PATCH] D82525: [FPEnv] Intrinsics for access to FP control modes

[Serge Pavlov via Phabricator via llvm-commits]

sepavloff added a comment.

In D82525#2605442 <https://reviews.llvm.org/D82525#2605442>, @craig.topper wrote:

> In D82525#2605358 <https://reviews.llvm.org/D82525#2605358>, @sepavloff wrote:
>
>> In D82525#2603611 <https://reviews.llvm.org/D82525#2603611>, @craig.topper wrote:
>>
>>> In D82525#2603275 <https://reviews.llvm.org/D82525#2603275>, @sepavloff wrote:
>>>
>>>> In D82525#2600598 <https://reviews.llvm.org/D82525#2600598>, @craig.topper wrote:
>>>>
>>>>> Is there any guarantee that femode_t will be the same layout for a given target in different C library implementations?
>>
>> There are cases when size of `fenv_t` differs in different libraries. ARM uses `unsigned int` in glibc but `unsigned long` in musl.
>
> Is `unsigned long` 32-bits in this case?

Yes, ARM gcc 10.2(linux) generates 4 for `sizeof(unsigned long)`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82525/new/

https://reviews.llvm.org/D82525