[PATCH] D103807: [clang][deps] Ensure deterministic order of TU '-fmodule-file=' arguments

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jun 13 13:24:39 PDT 2021


dexonsmith added a comment.

I'm not sure the performance problems with std::map will matter in practice here, but have you considered sorting before emission rather than relying on the data structure's iteration order? (It'd make it easy to switch to StringMap in the future.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103807



More information about the cfe-commits mailing list