[PATCH] D142041: [RISCV] Use TargetMachine MCSubtargetInfo in RISCVAsmPrinter::EmitHwasanMemaccessSymbols.

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 18 10:39:33 PST 2023


reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.

LGTM w/comment addressed.



================
Comment at: llvm/lib/Target/RISCV/RISCVAsmPrinter.cpp:264
   assert(TM.getTargetTriple().isOSBinFormatELF());
+  const MCSubtargetInfo &MCSTI = *TM.getMCSubtargetInfo();
 
----------------
Add a comment which highlights why we want the TM version here please.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142041



More information about the llvm-commits mailing list