[PATCH] D82988: [RISCV] Avoid Splitting MBB in RISCVExpandPseudo
    Luís Marques via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Jul  1 12:27:09 PDT 2020
    
    
  
luismarques added a comment.
This is looking quite good. As Sam says, not splitting BBs is preferable, so while the D79635 <https://reviews.llvm.org/D79635> issues could be resolved in other ways, if we can make it work like this then that's even better.
I suppose that this patch could also remove everything related to `LabelMustBeEmitted`.
If anyone wants to provide about the status of `setPreInstrSymbol` that would be helpful. Currently it's marked as `FIXME: This is not fully implemented yet.`, although Sam indicates that it's being used by x86, so that may be just a holdover.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82988/new/
https://reviews.llvm.org/D82988
    
    
More information about the llvm-commits
mailing list