skan marked an inline comment as done. skan added inline comments.
================ Comment at: llvm/lib/MC/MCFragment.cpp:426 + case MCFragment::FT_MachineDependent: { + const MCMachineDependentFragment *MF = + cast<MCMachineDependentFragment>(this); ---------------- MaskRay wrote: > `const auto *`. The type is obvious according to the right hand side. Shall we keep consistent with the local code style? `const MCLEBFragment *LF = cast<MCLEBFragment>(this);` was used here. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70157/new/ https://reviews.llvm.org/D70157 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits