[PATCH] D145765: Add __builtin_set_flt_rounds
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Mar 14 06:55:24 PDT 2023
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
In D145765#4191800 <https://reviews.llvm.org/D145765#4191800>, @xiongji90 wrote:
> I have simplified the test as you suggested and I also updated release doc to add "__builtin_set_flt_rounds" in fp support section. To `LanguageExtension.rst`, I prefer to creating a patch since we already had another builtin "__builtin_flt_rounds" which was not documented, I plan to document these 2 builtin together.
Documenting the two builtins together in a follow-up patch sounds fine to me, thank you!
LGTM
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D145765/new/
https://reviews.llvm.org/D145765
More information about the cfe-commits
mailing list