[PATCH] D58356: Hexagon: Add ImmArg to intrinsics

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 18 12:03:37 PST 2019


arsenm added a comment.

In D58356#1401381 <https://reviews.llvm.org/D58356#1401381>, @arsenm wrote:

> > Was the S2_insert omitted because you missed it when you were editing the .td file, or because your testing didn't find it?  From your description so far, all I really know is that you added an assertion somewhere, the rest is really vague.
>
> I didn't find it, I'll try to figure out why.


S2_insert is covered, it's just handled inside Hexagon_i32_i32i32i32i32_Intrinsic


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

https://reviews.llvm.org/D58356





More information about the llvm-commits mailing list