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

Tim Northover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 29 12:53:16 PDT 2017


t.p.northover added a comment.

> It's entirely possible that the intermediate instructions may fail legalization (for e.g. missing LegalizeActions/custom lowering of other intrinsics etc).

And then reporting an error on those would be fine, surely? I don't see why we have to defer deleting an instruction by passing a weird success code up the stack.


Repository:
  rL LLVM

https://reviews.llvm.org/D31359





More information about the llvm-commits mailing list