[PATCH] D66164: [LifetimeAnalysis] Support std::stack::top() and std::optional::value()

Gábor Horváth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 13 15:04:59 PDT 2019


xazax.hun added inline comments.


================
Comment at: clang/lib/Sema/SemaInit.cpp:6583
         .Cases("end", "rend", "cend", "crend", true)
-        .Cases("c_str", "data", "get", true)
+        .Cases("c_str", "data", "get", "value", true)
         // Map and set types.
----------------
Oh, this one needs to be updated, as `value` returns a reference not a pointer. 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66164/new/

https://reviews.llvm.org/D66164





More information about the cfe-commits mailing list