[PATCH] D63494: [AMDGPU] Fix for branch offset hardware workaround

[Ryan Taylor via Phabricator via llvm-commits]

rtaylor marked an inline comment as done.
rtaylor added inline comments.


================
Comment at: lib/Target/AMDGPU/SOPInstructions.td:974
+  def "" : SOPP<op, ins, asm, pattern>;
+  def _PAD_S_NOP : SOPP64<op, ins, asm, pattern>;
+}
----------------
rampitec wrote:
> rtaylor wrote:
> > rampitec wrote:
> > > Can yo use InstrMapping mapping here to avoid switches?
> > Do you feel that this is worth it in this case? Seems overly complicated for a very specific use case but if it's the standard to go in this direction than ok. 
> You just get rid of one switch, so it is not that big deal now.
That's ok, I've pretty much done it.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63494/new/

https://reviews.llvm.org/D63494