[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