[PATCH] D34160: [Power9] Exploit vinserth instruction
Tony Jiang via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 16 07:55:52 PDT 2017
jtony added inline comments.
================
Comment at: lib/Target/PowerPC/PPCISelLowering.cpp:1629
+ uint32_t Mask = 0;
+ uint32_t OriginalOrderLow = 0x1234567;
+ uint32_t OriginalOrderHigh = 0x89ABCDEF;
----------------
Is there any reason why we use uint32_t? If not, I would use unsigned instead to make it consistent. We are using both unsigned and uint32_t in this file.
Repository:
rL LLVM
https://reviews.llvm.org/D34160
More information about the llvm-commits
mailing list