[PATCH] D111709: [Flang] flang-omp-report replace std::vector's with llvm::SmallVector

David Truby via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 13 05:53:58 PDT 2021


DavidTruby added a comment.

LGTM, wait for another review though



================
Comment at: flang/examples/flang-omp-report-plugin/flang-omp-report-visitor.h:101
+  llvm::SmallVector<OmpWrapperType *> ompWrapperStack;
+  std::map<OmpWrapperType *, llvm::SmallVector<ClauseInfo>> clauseStrings;
   Parsing *parsing{nullptr};
----------------
just putting this here as a note (this should be done in a separate patch):
This `std::map` should probably be changed to an `llvm::DenseMap` or similar


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111709



More information about the llvm-commits mailing list