[PATCH] D139608: [Clang][NFC] Add default `getBFloat16Mangling` impl
Ties Stuij via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 10 08:14:57 PST 2023
stuij added a comment.
I went back and forth on this for a bit. There's a review outstanding to change the mangling name for X86: https://reviews.llvm.org/D136919 to align with the C++23 std::bfloat16_t issue from above. I settled on preferring to be conservative, and not risk at some point accidentally opting an arch into a name that doesn't conform with their ABI.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139608/new/
https://reviews.llvm.org/D139608
More information about the cfe-commits
mailing list