[PATCH] D132482: RISCV: permit unaligned nop-slide padding emission
    Saleem Abdulrasool via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Aug 23 12:45:49 PDT 2022
    
    
  
compnerd marked 4 inline comments as done.
compnerd added inline comments.
================
Comment at: llvm/lib/Target/RISCV/MCTargetDesc/RISCVAsmBackend.cpp:363
+      OS.write("\x01\0", 2);
+
+  // If the count is not 4-byte aligned (or 2-byte under the C extensions), we
----------------
jrtc27 wrote:
> Making this a loop is a bit odd when we know it's always executing 0 or 1 times
It seems nicer and feels more in line with the non-C version.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132482/new/
https://reviews.llvm.org/D132482
    
    
More information about the llvm-commits
mailing list