[clang] 30f0cd5 - [Sema] followups from efe4a54884cb

Sam McCall via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 18 04:33:49 PDT 2023


Author: Sam McCall
Date: 2023-08-18T13:33:30+02:00
New Revision: 30f0cd55c614639ad5af7b32846cd4fea75d968e

URL: https://github.com/llvm/llvm-project/commit/30f0cd55c614639ad5af7b32846cd4fea75d968e
DIFF: https://github.com/llvm/llvm-project/commit/30f0cd55c614639ad5af7b32846cd4fea75d968e.diff

LOG: [Sema] followups from efe4a54884cb

Accidentally left review fixes out of the commit.

Added: 
    

Modified: 
    clang/include/clang/Sema/Ownership.h

Removed: 
    


################################################################################
diff  --git a/clang/include/clang/Sema/Ownership.h b/clang/include/clang/Sema/Ownership.h
index 83b9c9a3652a8e..0752f5de7e3342 100644
--- a/clang/include/clang/Sema/Ownership.h
+++ b/clang/include/clang/Sema/Ownership.h
@@ -144,15 +144,15 @@ template <class Ptr> struct IsResultPtrLowBitFree {
 /// The result of parsing/analyzing an expression, statement etc.
 ///
 /// It may be:
-/// - a valid pointer to the result object
-/// - unset (null but valid), for constructs that may legitimately be absent
+/// - usable: a valid pointer to the result object
+/// - unset (null but valid): for constructs that may legitimately be absent
 ///   (for example, the condition of a for loop)
-/// - invalid, indicating an error
+/// - invalid: indicating an error
 ///   (no detail is provided, usually the error has already been diagnosed)
 template <class PtrTy, bool Compress = IsResultPtrLowBitFree<PtrTy>::value>
 class ActionResult {
   PtrTy Val = {};
-  bool Invalid;
+  bool Invalid = false;
 
 public:
   ActionResult(bool Invalid = false) : Val(PtrTy()), Invalid(Invalid) {}


        


More information about the cfe-commits mailing list