[PATCH] D149838: [clang][dataflow] Eliminate `SkipPast::ReferenceThenPointer`.

Martin Böhme via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 12 05:00:11 PDT 2023


mboehme marked an inline comment as done.
mboehme added inline comments.


================
Comment at: clang/lib/Analysis/FlowSensitive/DataflowEnvironment.cpp:840
+    return nullptr;
+  } else {
+    return cast<AggregateStorageLocation>(Loc);
----------------
ymandel wrote:
> nit: no need for `else` since `if` branch always returns
Good point -- done. (Also below.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149838



More information about the cfe-commits mailing list