[PATCH] D149633: [clang][codegen] Add F128 vsnprintf_chk builtin

Tulio Magno Quites Machado Filho via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 2 08:27:11 PDT 2023


tuliom requested changes to this revision.
tuliom added a comment.
This revision now requires changes to proceed.

@tbaeder Keep in mind that I don't think this will completely fix issue 61913 because there are other 2 functions affected.
Otherwise, I believe this patch is in the right direction.



================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:104
       {Builtin::BI__builtin_printf, "__printfieee128"},
+      {Builtin::BI__builtin___vsnprintf_chk, "__vsnprintfieee128"},
       {Builtin::BI__builtin_vsnprintf, "__vsnprintfieee128"},
----------------
I believe this should be redirected to `__vsnprintf_chkieee128`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149633



More information about the cfe-commits mailing list