[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