[spirv] Restoring breaks incorrectly dropped

[maarquitos14]

Breaks were lost in 5fd4877, restoring them. When the breaks are lost, literals are enconded/decoded twice, which is wrong.

** This is done in this repo because this piece of code is not present in Khronos repo. **

[vmaksimo]

Nice catch!

[maarquitos14]

@intel/llvm-gatekeepers I think this is ready to merge.

[sommerlukas]

@maarquitos14 Can you please check your Github account settings to include your actual mail address in the commit for PRs:

This commit will be authored by [email protected]

We can only merge after this is fixed.

[maarquitos14]

@maarquitos14 Can you please check your Github account settings to include your actual mail address in the commit for PRs:

This commit will be authored by [email protected]

We can only merge after this is fixed.

This is very weird, I have had several PRs merged in the last year and a half without doing this. Did anything change in Github?

[sommerlukas]

Not that I'm aware of, but maybe this was just missed on previous PRs.

The note to gatekeepers to look out for this is relatively new (as in a few months old).