[PATCH] D95400: [flang][fir] Upstream FIR support changes

[Kiran Chandramohan via Phabricator via llvm-commits]

kiranchandramohan added a comment.

A few minor comments.



================
Comment at: flang/include/flang/Optimizer/Support/FIRContext.h:12
+//===----------------------------------------------------------------------===//
+/// \file
+///
----------------
Is this stray or a directive to some tool?


================
Comment at: flang/include/flang/Optimizer/Support/InternalNames.h:44
+    VARIABLE,
+    BLOCK_DATA_NAME
   };
----------------
Nit: Was this previously in alphabetical order?


================
Comment at: flang/lib/Optimizer/Support/InternalNames.cpp:102
 
+std::string fir::NameUniquer::doBlockData(llvm::StringRef name) {
+  std::string result = prefix();
----------------
Did we have tests for all these?


================
Comment at: flang/lib/Optimizer/Support/KindMapping.cpp:310
+KindTy fir::KindMapping::defaultDoubleKind() const {
+  auto iter = defaultMap.find('c');
+  assert(iter != defaultMap.end());
----------------
Should this be 'd'?


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

https://reviews.llvm.org/D95400