[PATCH] D128159: [DAG] Enable scalable vectors handling in computeKnownBits
    Simon Pilgrim via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Jun 20 01:23:44 PDT 2022
    
    
  
RKSimon added a comment.
SGTM but I'll defer to people who are actively working with scalable vectors whether the getVectorMinNumElements approach will work.
I had been wondering whether we're going have to move from APInt to some kind of 'DemandedMask' wrapper class to handle everything,
================
Comment at: llvm/lib/Target/AArch64/AArch64InstrFormats.td:61
+  }];
+}
+
----------------
Just move this as a pre-commit NFC?
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128159/new/
https://reviews.llvm.org/D128159
    
    
More information about the llvm-commits
mailing list