[PATCH] [AArch64] Support ISD::SIGN_EXTEND_INREG

Jiangning Liu liujiangning1 at gmail.com
Sun Jan 12 19:22:58 PST 2014


Hi 
Ana,

I have two reasons for handling this situation in "combine"
 stage.

1) If we customize lowering sext_inreg, we would probably missing the case
of explicitly using shl/ashr in LLVM IR. I added three more cases with
suffix "_2" to cover this kind of test. If we want to cover all in
legalization, we would have to write more codes, I think.
2) LLVM documentation says "Subsequent runs of the pass clean up the messy
code generated by the Legalize passes, which allows Legalize to be very
simple (it can focus on making code legal instead of focusing on generating
good and legal code)."

Kevin already upstreamed the improvement of generating permutation
instructions, so now I'm uploading new patches by
1) Checking uzip1 in the test
2) Adding three more cases for explicity shl/ashr pair.

Thanks,
-Jiangning
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140113/316fc921/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: sext_inreg_llvm_4.patch
Type: application/octet-stream
Size: 13905 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140113/316fc921/attachment.obj>


More information about the llvm-commits mailing list