stefanp-ibm wrote:
> I personally prefer this solution. @stefanp-ibm What do you think?
>
> LGTM
I understand your point about forcing users to just use `mffs` when `mffsl` is
not available. If that's the case do we also want to change this:
```
#ifdef _ARCH_PWR9
__fpscr_save.__fr = __builtin_ppc_mffsl();
#else
__fpscr_save.__fr = __builtin_ppc_mffs();
__fpscr_save.__fpscr &= 0x70007f0ffL;
#endif
```
and just use:
```
#ifdef _ARCH_PWR9
__fpscr_save.__fr = __builtin_ppc_mffsl();
#else
// Since mffsl is not available on anything before Power 9 use the longer
latency instruction.
__fpscr_save.__fr = __builtin_ppc_mffs();
#endif
```
https://github.com/llvm/llvm-project/pull/67299
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits