[PATCH] D67649: [FileCheck] Move private interface to its own header

[Alexander Richardson via Phabricator via llvm-commits]

arichardson added inline comments.


================
Comment at: llvm/include/llvm/Support/FileCheck.h:147
+  //CXX17 TODO: make this a std::vector.
+  std::vector<FileCheckString> *CheckStrings;
 
----------------
Why can't this be by-vaule? Shouldn't a vector of an incomplete type be fine as long as you have an out-of-line ctor+dtor?


================
Comment at: llvm/lib/Support/FileCheck.cpp:1726
+FileCheck::~FileCheck() {
+  delete PatternContext;
+  delete CheckStrings;
----------------
Use `unique_ptr` instead of new delete?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67649