[PATCH] D97462: [clang][cli] Round-trip cc1 arguments in assert builds

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 2 11:42:30 PDT 2021


dexonsmith added a comment.

In D97462#2666485 <https://reviews.llvm.org/D97462#2666485>, @jansvoboda11 wrote:

> Thanks for reporting that. D99606 <https://reviews.llvm.org/D99606> fixes one aspect of `-plugin-arg`, but it seems the order of generation is non-deterministic (most likely related to the underlying storage, `std::unordered_map`). I can look into it early next week, but I think simple sort in the generation code should do the trick.

Can/should it just be changed to a `std::map`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97462



More information about the cfe-commits mailing list