[PATCH] D92010: [clang-offload-bundler] use std::forward_list for storing temp file names [NFC]

Sergey Dmitriev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 24 06:55:55 PST 2020


sdmitriev added inline comments.


================
Comment at: clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp:411
 private:
-  SmallVector<SmallString<128u>, 4u> Files;
+  std::forward_list<SmallString<128u>> Files;
 };
----------------
ABataev wrote:
> What about `llvm::iplist`?
It can be changed to iplist as well, but why do you think it is a better option here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92010



More information about the cfe-commits mailing list