nemanjai accepted this revision.
nemanjai added a comment.
This revision is now accepted and ready to land.
LGTM as long as the sign-extending test is added.
================
Comment at:
llvm/test/CodeGen/PowerPC/builtins-ppc-xlcompat-load-store-reversed.ll:52
define dso_local zeroext i16 @test_builtin_ppc_load2r() {
; CHECK-64B-LABEL: test_builtin_ppc_load2r:
----------------
Please add an equivalent test with `signext` to ensure there is a `extsh` prior
to the return.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110771/new/
https://reviews.llvm.org/D110771
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
