[PATCH] D110771: [PowerPC] Fix __builtin_ppc_load2r to return short instead of int.
Nemanja Ivanovic via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Sep 30 18:23:05 PDT 2021
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
More information about the cfe-commits
mailing list