[PATCH] D77082: [AMDGPU] Handle SMRD signed offset immediate

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added a comment.

Please follow lint recommendations on formatting.



================
Comment at: llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp:1270
 
-Optional<int64_t> getSMRDEncodedOffset(const MCSubtargetInfo &ST,
-                                       int64_t ByteOffset) {
+Optional<int64_t> getSMRDEncodedUnsignedOffset(const MCSubtargetInfo &ST,
+                                               int64_t ByteOffset) {
----------------
Should we instead pass here Subtarget and distinguish if it has needed mode right in this function?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77082