[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