[PATCH] D138352: [RISCV] Support .variant_cc directive for the assembler.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 2 10:08:37 PST 2022


MaskRay requested changes to this revision.
MaskRay added inline comments.
This revision now requires changes to proceed.


================
Comment at: llvm/lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp:2312
+  StringRef SymbolName = Tok.getIdentifier();
+  MCSymbol *Sym = getContext().lookupSymbol(SymbolName);
+  if (!Sym)
----------------
`MCSymbol *Sym = getContext().lookupSymbol(SymbolName);` is not the best. See D122507 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138352



More information about the llvm-commits mailing list