[PATCH] D75542: [Sema] Prevent UB for uninitialized `IsSurrogate`
Brian Gesiak via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 13 07:28:35 PDT 2020
modocache abandoned this revision.
modocache added a comment.
Awesome, thanks!
> Alternatively, I considered modifying the `clang::OverloadCandidate` constructor to initialize `IsSurrogate` with a false value. I feel doing so would be safer, but I stuck with what appears to be the convention: initializing OverloadCandidate members are the site of use.
Looks like the patch does exactly this, which is nice.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75542/new/
https://reviews.llvm.org/D75542
More information about the cfe-commits
mailing list