[PATCH] D95675: [AMDGPU][MC] Corrected parsing of optional modifiers

Dmitry Preobrazhensky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 29 06:17:09 PST 2021


dp added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp:5933
+    StringRef Tok = getTokenStr();
+    if (Tok.startswith(Pref) && Tok.substr(Pref.size()) == Id) {
+      lex();
----------------
foad wrote:
> drop_front might be a bit clearer than substr?
Thanks! I'll correct this before committing.


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

https://reviews.llvm.org/D95675



More information about the llvm-commits mailing list