[llvm] MC: Store MCRelaxableFragment MCInst out-of-line (PR #147229)

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 7 21:21:29 PDT 2025


================
@@ -431,16 +428,38 @@ class MCDataFragment : public MCEncodedFragment {
 ///
 class MCRelaxableFragment : public MCEncodedFragment {
   /// The instruction this is a fragment for.
-  MCInst Inst;
----------------
MaskRay wrote:

You are right. I take back my previous comment.. Updated the description

> Unfortunately, we also have to encode MCInst::Flags to support the EVEX prefix, e.g. `{evex} xorw $foo, %ax`.

I only noticed the case after adding the `getFlags() == 0` assertion... The #78545 case should probably switch to llvm-objdump -dr

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


More information about the llvm-commits mailing list