tree a94b7c45a33d5bfadf64d0d94891ad47463b790e
parent 8915a7c8f120b5d75b915998f75a2fbbc4adf0b3
author Steven Perron <stevenperron@google.com> 1548359458 -0500
committer GitHub <noreply@github.com> 1548359458 -0500
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJcShciCRBK7hj4Ov3rIwAAdHIIADdeIaXf2gh7gRBcCj44m3ak
 OgbjFwLqlYE/biC/gocJmBrGpRrpxbtPfPjU+lBwKkmuhGfxyjsCZZTRq2WLd/vG
 K3PtRwlMlX6ixEflEw6sz2oCz54OG5MZJUB6nHyxQaTq0xeGsDePfpO6zliXh1+x
 t7f+w/BTeNwkkNnuJVRU3go5rQuvA7aSgHD2+2/VD/hb+qwNOqcFOBSoE0G2vpxK
 eA8QrEq3834vTmjBrHbp9a6q+jIseaXiKPMQvcJ44IkLr6ntOdvKAyHha1Pr/NHS
 3b/YQcDbHuL2X1Lq1CjFM4AHSB9m1ige9klKWLjRv54fEKfyfF2MdT8e+AsmnTg=
 =IRX8
 -----END PGP SIGNATURE-----
 

Remove the static maps from CheckDecorationsCompatibility (#2327)

* Remove the static maps from CheckDecorationsCompatibility

There are a few data structures in the function
`CheckDecorationsCompatibility` that are allocated using `new` and their
address is stored in a static pointer.  This code pattern causes the
MSVC memory leak checker to say there is a memory leak.  Some people
are interested in keeping that clean.

To work around it, I have replaced them with either a function or an
array of POD types.  The array can be kept as a static directly because
it has a trivial destructor, and we don't have to worry about it being
destroyed too early.

Fixes https://github.com/KhronosGroup/SPIRV-Tools/issues/2317.