[lld] [llvm] [ICF] Add a NOP after branch in ICF thunk to improve debugability (PR #154986)

[David Blaikie via llvm-commits]

dwblaikie wrote:

So far as I know, one instruction sequences are fine - /zero/ instruction sequences are problematic.

Is this about one instruction, or zero instruction? Could you say more about what problems one instruction sequences have?

I had trouble following info on the linked PR #154851 - it says things like "LowPC == HighPC" - but that shouldn't be the case for a single instruction sequence, right? (I'd have expected: LowPC + sizeof(instruction) + 1 == HighPC)

https://github.com/llvm/llvm-project/pull/154986