[llvm] [MC] Add and use const overload of MCAsmParser::getContext. NFC. (PR #160490)

via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 24 03:26:17 PDT 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-backend-amdgpu

Author: Jay Foad (jayfoad)

<details>
<summary>Changes</summary>



---
Full diff: https://github.com/llvm/llvm-project/pull/160490.diff


3 Files Affected:

- (modified) llvm/include/llvm/MC/MCParser/MCAsmParser.h (+1) 
- (modified) llvm/include/llvm/MC/MCParser/MCAsmParserExtension.h (+1) 
- (modified) llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp (+1-3) 


``````````diff
diff --git a/llvm/include/llvm/MC/MCParser/MCAsmParser.h b/llvm/include/llvm/MC/MCParser/MCAsmParser.h
index e3f44a08db641..ea0c29e16c397 100644
--- a/llvm/include/llvm/MC/MCParser/MCAsmParser.h
+++ b/llvm/include/llvm/MC/MCParser/MCAsmParser.h
@@ -162,6 +162,7 @@ class LLVM_ABI MCAsmParser {
   virtual void addAliasForDirective(StringRef Directive, StringRef Alias) = 0;
 
   MCContext &getContext() { return Ctx; }
+  const MCContext &getContext() const { return Ctx; }
   MCStreamer &getStreamer() { return Out; }
   SourceMgr &getSourceManager() { return SrcMgr; }
   AsmLexer &getLexer() { return Lexer; }
diff --git a/llvm/include/llvm/MC/MCParser/MCAsmParserExtension.h b/llvm/include/llvm/MC/MCParser/MCAsmParserExtension.h
index f8387d888fc21..5344acd8012d8 100644
--- a/llvm/include/llvm/MC/MCParser/MCAsmParserExtension.h
+++ b/llvm/include/llvm/MC/MCParser/MCAsmParserExtension.h
@@ -53,6 +53,7 @@ class LLVM_ABI MCAsmParserExtension {
   /// @{
 
   MCContext &getContext() { return getParser().getContext(); }
+  const MCContext &getContext() const { return getParser().getContext(); }
 
   AsmLexer &getLexer() { return getParser().getLexer(); }
   const AsmLexer &getLexer() const {
diff --git a/llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp b/llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
index 56f79c2d67d3f..df3070b92e32c 100644
--- a/llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
+++ b/llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
@@ -1592,9 +1592,7 @@ class AMDGPUAsmParser : public MCTargetAsmParser {
   }
 
   const MCRegisterInfo *getMRI() const {
-    // We need this const_cast because for some reason getContext() is not const
-    // in MCAsmParser.
-    return const_cast<AMDGPUAsmParser*>(this)->getContext().getRegisterInfo();
+    return getContext().getRegisterInfo();
   }
 
   const MCInstrInfo *getMII() const {

``````````

</details>


https://github.com/llvm/llvm-project/pull/160490


More information about the llvm-commits mailing list