efriedma added a comment.

I'd prefer to handle legalization in a separate patch from handling legal 
sdiv/udiv operations, so we actually have some context to discuss the 
legalization strategy.



================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:7670
+                     Mask, Op.getOperand(0), Op.getOperand(1));
+}
+
----------------
If we're going to support these operations, we might as well just add isel 
patterns; that's what we've been doing for other arithmetic operations.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78569/new/

https://reviews.llvm.org/D78569



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to