[PATCH] D70157: Align branches within 32-Byte boundary
    Kan Shengchen via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Mon Dec 16 17:21:49 PST 2019
    
    
  
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
    
    
More information about the cfe-commits
mailing list