[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