================
@@ -3840,6 +3840,56 @@ TEST_P(UncheckedStatusOrAccessModelTest,
NestedStatusOrInStatusOrStruct) {
)cc");
}
+TEST_P(UncheckedStatusOrAccessModelTest, StatusOrPtrReference) {
+ ExpectDiagnosticsFor(R"cc(
+#include "unchecked_statusor_access_test_defs.h"
+
+ const STATUSOR_INT* foo();
+
+ void target() {
+ const auto& sor = foo();
+ if (sor->ok()) sor->value();
+ }
+ )cc");
+
+ ExpectDiagnosticsFor(R"cc(
+#include "unchecked_statusor_access_test_defs.h"
+
+ using StatusOrPtr = const STATUSOR_INT*;
+ StatusOrPtr foo();
+
+ void target() {
+ const auto& sor = foo();
----------------
fmayer wrote:
yes. this PR is about references to Status(Or) pointers. Maybe I should make
the commit message clear
https://github.com/llvm/llvm-project/pull/170951
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
