[PATCH] D92492: [flang] Fix bugs related to merging generics during USE

[Tim Keith via Phabricator via llvm-commits]

tskeith added inline comments.


================
Comment at: flang/lib/Semantics/CMakeLists.txt:1
+set(CMAKE_CXX_FLAGS_RELEASE_G1_O0 "-g1 -O0")
+set(CMAKE_BUILD_TYPE RELEASE_G1_O0)
----------------
klausler wrote:
> Should these remain?
That was left over from debugging. I'll get rid of it.


================
Comment at: flang/lib/Semantics/resolve-names.cpp:2417
+  } else if (auto *genericDetails{localSymbol.detailsIf<GenericDetails>()}) {
+    if (auto *useDetails{useSymbol.detailsIf<GenericDetails>()}) {
+      AddGenericUse(*genericDetails, localName, useSymbol);
----------------
klausler wrote:
> clang-tidy's suggested `const auto *` seems good to me here.
I'll fix those.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92492