[PATCH] D56935: [NewPM] Add support for new-PM plugins to clang

[Philip Pfaffe via Phabricator via cfe-commits]

philip.pfaffe added a comment.

This generally looks sane. What will happen on windows though? Will it silently fail?



================
Comment at: clang/include/clang/Basic/CodeGenOptions.h:292
+  /// List of dynamic shared object files to be loaded as pass plugins.
+  std::vector<std::string> PassPlugins;
+
----------------
This  should be SmallVector.


Repository:
  rC Clang

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

https://reviews.llvm.org/D56935