[PATCH] D158066: [PowerPC] Fix use of FPSCR builtins in smmintrin.h

qiucf abandoned this revision. qiucf added a comment. Migrated to https://github.com/llvm/llvm-project/pull/67299 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158066/new/ https://reviews.llvm.org/D158066 __

[PATCH] D158066: [PowerPC] Fix use of FPSCR builtins in smmintrin.h

qiucf updated this revision to Diff 555805. qiucf added a comment. - Add macro alias to `mffs` `mffsl` `mtfsf` and `set_fpscr_rn`, although they don't work in freestanding mode - Add C++ run lines to intrinsics tests. To avoid further messing codegen checks, make them run under `-fsyntax-only`.

[PATCH] D158066: [PowerPC] Fix use of FPSCR builtins in smmintrin.h

qiucf added a comment. In D158066#4601785 , @nemanjai wrote: > It should be perfectly fine to provide pre-defined macros for these to match > GCC on PowerPC. The reason we went with the macro solution is to avoid > polluting the builtins namespace for o

[PATCH] D158066: [PowerPC] Fix use of FPSCR builtins in smmintrin.h

nemanjai added a comment. In D158066#4593961 , @qiucf wrote: > CC @amyk @quinnp Any comments about the naming? > > I see some `__builtin_ppc_xxx` are aliased into `__builtin_xxx` by > `defineXLCompatMacros`. But these are not XL-compatible builtins, and

[PATCH] D158066: [PowerPC] Fix use of FPSCR builtins in smmintrin.h

qiucf added subscribers: quinnp, amyk. qiucf added a comment. CC @amyk @quinnp Any comments about the naming? I see some `__builtin_ppc_xxx` are aliased into `__builtin_xxx` by `defineXLCompatMacros`. But these are not XL-compatible builtins, and the macros do not always work. Repository: r

[PATCH] D158066: [PowerPC] Fix use of FPSCR builtins in smmintrin.h

tuliom added a comment. Are there any plans to align the names of these built-ins with GCC? Their built-ins do not have the `_ppc` part. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158066/new/ https://reviews.llvm.org/D158066 __

[PATCH] D158066: [PowerPC] Fix use of FPSCR builtins in smmintrin.h

qiucf created this revision. qiucf added reviewers: nemanjai, shchenz, stefanp, PowerPC. Herald added a subscriber: kbarton. Herald added a project: All. qiucf requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. `smmintrin.h` uses `__builtin_mf