xiongji90 added a comment.

In D145765#4189153 <https://reviews.llvm.org/D145765#4189153>, @aaron.ballman 
wrote:

> Thanks for the additional tests! Can you also add a release note to 
> `clang/docs/ReleaseNotes.rst` and document the new builtin in 
> `clang\docs\LanguageExtensions.rst`?

Hi, @aaron.ballman 
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.
Thanks very much.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145765

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to