[PATCH] D10303: Don't use ManagedStatic.
Rafael Avila de Espindola via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 15 14:16:51 PDT 2018
espindola accepted this revision.
espindola added a comment.
This revision is now accepted and ready to land.
I agree that using function local static would be an improvement regardless of any speedup.
According to https://msdn.microsoft.com/en-au/library/hh567368.aspx#concurrencytable this works with MSVC 2015 and according to http://llvm.org/docs/GettingStartedVS.html we now required at least 2015.
LGTM witch a local static instead of a mutex.
https://reviews.llvm.org/D10303
More information about the llvm-commits
mailing list