[PATCH] D105461: [Support] CommandLine.cpp - Fix thread race condition in addOption
Aaron Ballman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 8 03:54:36 PDT 2021
aaron.ballman accepted this revision.
aaron.ballman added reviewers: hokein, rnk.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
Thanks for the fix! This LGTM as far as the change goes, though I wonder if there are other race conditions hiding in this file. I'm not an expert in this particular area, so you should wait a few days before landing in case any of the other reviewers have comments (I added a few more folks who have touched this file in recent history).
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105461/new/
https://reviews.llvm.org/D105461
More information about the llvm-commits
mailing list