[PATCH] D21807: [Mips] Implement shouldSignExtI32Param hook.

Daniel Sanders via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 29 02:35:43 PDT 2016


dsanders accepted this revision.
dsanders added a comment.
This revision is now accepted and ready to land.

As I mentioned on the original patch, the parameter side of it looks correct but the return side looks wrong to me. However, you're right that the returns match the current behaviour of clang so I'll give this an LGTM and if my suspicion about clang's behaviour being wrong turns out to be correct then we can fix both of these at the same time.


Repository:
  rL LLVM

http://reviews.llvm.org/D21807





More information about the llvm-commits mailing list