[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