tree 1cca3c91fcf8b9667c03c832c4b0ca386d6a5179
parent 3a252a267b065b66ac2af8aa29f125d71fd3ce1b
author Steven Perron <stevenperron@google.com> 1562773036 -0400
committer GitHub <noreply@github.com> 1562773036 -0400
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJdJgYsCRBK7hj4Ov3rIwAAdHIIAHWcc9OfvT9dl+9oORkC4Eib
 CAIY24t5YkIkuA0c4hCWaXb7N0dZpGEsWssU5N0h/2JtsP4zEpRXM8scV1AA8zJV
 wv59y7jCFkI4bSlUtzZMnBLxVn6ToJub7+bJqJv12dulO0H9PQ8JFOxuL7aOeIs7
 357kUO8qJm+VZhoqAs8aNGSe09fo/1z+52Ckm07Qqx/x78izsKure7nEw8eM3Sbb
 pBSNGfYSPhAfXv9pdOcEk2NLUkKJbXhrUiFVIaMmQxPjIqe5RM+D5Bja7tX+XVd8
 KJWw1P/NTpeo3O9Y1Kt+f1Ydq+YfY/JZsC5SRye9srGOV6Mp0amesHaDkCAxT0U=
 =lBba
 -----END PGP SIGNATURE-----
 

Handle decorations better in some optimizations (#2716)

There are a couple spots where we are not looking at decorations when we should.

1. Value numbering is suppose to assign a different value number to ids if they have different decorations.  However that is not being done for OpCopyObject and OpPhi.

1. Instruction simplification is propagating OpCopyObject instruction without checking for decorations.  It should only do that if no decorations are being lost.

Add a new function to the decoration manager to check if the decorations of one id are a subset of the decorations of another.

Fixes #2715.