craig.topper added a comment.
In D133634#3795044 <https://reviews.llvm.org/D133634#3795044>, @erichkeane
wrote:
> The CFE Changes look correct, but @eli.friedman/@craig.topper should comment
> as to whether this is acceptable.
This is acceptable to me.
================
Comment at: clang/lib/AST/MicrosoftMangle.cpp:3111
Extra.mangleSourceName("__vector");
- Extra.mangleType(QualType(ET, 0), Range, QMM_Escape);
+ Extra.mangleType(QualType(ET ? static_cast<const Type *>(ET) : BitIntTy,
0),
+ Range, QMM_Escape);
----------------
Why is an explicit cast needed here?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133634/new/
https://reviews.llvm.org/D133634
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
