================ @@ -232,6 +232,10 @@ Changes in existing checks <clang-tidy/checks/bugprone/unchecked-optional-access>` to support `bsl::optional` and `bdlb::NullableValue` from <https://github.com/bloomberg/bde>_. + Fixed false positives from smart pointer accessors repeated in checking + ``has_value`` and accessing ``value``, by caching the locations returned + by the accessors. The option `IgnoreSmartPointerDereference` should no + longer be needed. ---------------- jvoung wrote:
I think it can be removed. By "maybe could be removed", did you mean to actually remove the option (together with this change or as a separate change), or to clarify in the release note that it could be removed in the future/soon? I wasn't sure if it was okay to remove a flag before advertising that it isn't needed. (though It looks like right now clang-tidy just ignores options that aren't available, instead of failing with an error)? https://github.com/llvm/llvm-project/pull/122290 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
