[PATCH] D31359: [GlobalISel]: Allow backends to custom legalize Intrinsics

Aditya Nandakumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 27 17:33:01 PST 2019


aditya_nandakumar added a comment.

In D31359#1412300 <https://reviews.llvm.org/D31359#1412300>, @arsenm wrote:

> I'm not sure I understand why this needs to be separate from legalizeCustom


IIUC, intrinsics are defined with variable no of srcs and dsts and I'm not sure if it's possible for any legalization query to specify actions(custom) based on intrinsic id which would imply different types for different operands. Also, this patch was written a long time and at that time it wasn't possible to specify intrinsic ids + operands for legalization rules. Also, I'm not sure if it's' possible now.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D31359





More information about the llvm-commits mailing list