[PATCH] D76679: [SveEmitter] Add more immediate operand checks.
    Sander de Smalen via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Fri Apr  3 09:08:06 PDT 2020
    
    
  
sdesmalen added a comment.
In D76679#1957399 <https://reviews.llvm.org/D76679#1957399>, @SjoerdMeijer wrote:
> I think the float16 discussion is an interesting one, but doesn't necessarily need to be done here. I am asking some questions offline, but if we ever come to a different opinion on it, then we can follow up so it's somewhat orthogonal to this change, and so this looks fine to me.
Thanks for reviewing the patch Sjoerd!
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76679/new/
https://reviews.llvm.org/D76679
    
    
More information about the cfe-commits
mailing list