Clean up fgRemoveDeadStore.
Clean up fgRemoveDeadStore.

Remove duplication and a couple of goto's from `fgRemoveDeadStore`.

This part was very confusing:

`
                /* If this is GT_CATCH_ARG saved to a local var don't bother */

                JITDUMP("removing stmt with no side effects\n");

                if (asgNode->gtFlags & GTF_ORDER_SIDEEFF)
                {
                    if (rhsNode->gtOper == GT_CATCH_ARG)
                    {
                        goto EXTRACT_SIDE_EFFECTS;
                    }
                }
`

The `goto` was to the preceding `if` and was useless since the call
to `gtExtractSideEffList(rhsNode, &sideEffList);` couldn't possibly find
side effects because we already checked that
`rhsNode->gtFlags & GTF_SIDE_EFFECT == 0`.

krk/coreclr,poizan42/coreclr,poizan42/coreclr,cshung/coreclr,krk/coreclr,cshung/coreclr,krk/coreclr,wtgodbe/coreclr,poizan42/coreclr,wtgodbe/coreclr,krk/coreclr,cshung/coreclr,poizan42/coreclr,wtgodbe/coreclr,wtgodbe/coreclr,cshung/coreclr,poizan42/coreclr,krk/coreclr,cshung/coreclr,poizan42/coreclr,krk/coreclr,cshung/coreclr,wtgodbe/coreclr,wtgodbe/coreclr