[PATCH] D117317: [MC] Put the pending fixups into relocation offset symbol's fragment

[luxufan via Phabricator via llvm-commits]

StephenFan marked an inline comment as done.
StephenFan added inline comments.


================
Comment at: llvm/lib/MC/MCObjectStreamer.cpp:129
+    switch (SymFragment->getKind()) {
+    case MCFragment::FT_Relaxable:
+    case MCFragment::FT_Dwarf:
----------------
Hi @MaskRay , The test case of FT_Relaxable is not added. Because I don't know how to define a symbol which its fragment is FT_Relaxable. Do you have some ideas ?


================
Comment at: llvm/lib/MC/MCObjectStreamer.cpp:137
+    case MCFragment::FT_Data:
+    case MCFragment::FT_CVDefRange:
+      cast<MCEncodedFragmentWithFixups<32, 4>>(SymFragment)
----------------
I found that FT_CVDefRange is only used by COFF. Does COFF support `.reloc` directive? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117317