For more details about this instruction, please refer to the latest ISE document: https://www.intel.com/content/www/us/en/develop/download/intel-architecture-instruction-set-extensions-programming-reference.html
Details
Diff Detail
- Repository
 - rG LLVM Github Monorepo
 
Event Timeline
| clang/lib/Headers/CMakeLists.txt | ||
|---|---|---|
| 207 | alphabetical order  | |
| llvm/include/llvm/IR/IntrinsicsX86.td | ||
| 5112 | DefaultAttrsIntrinsic  | |
| 5114 | ditto.  | |
| 5116 | ditto.  | |
| llvm/test/CodeGen/X86/sha512-intrinsics.ll | ||
| 2 | X64,CHECK  | |
| 2 | No need O0  | |
| 3 | X86,CHECK  | |
| 3 | ditto.  | |
| llvm/test/MC/Disassembler/X86/sha512-64.txt | ||
| 1–2 | This can be merged with sha512-32.txt  | |
| llvm/test/MC/X86/sha512-att-64.s | ||
| 1 | ditto.  | |
| llvm/test/MC/X86/sha512-intel-64.s | ||
| 1 | ditto.  | |
| llvm/lib/Target/X86/X86.td | ||
|---|---|---|
| 243 | AVX2 like other integer features?  | |
| llvm/lib/Target/X86/X86InstrSSE.td | ||
| 8304 | This paren should be indented 1 more space so that it's not at the same column as the one above it.  | |
| 8310 | ditto  | |
| 8316 | ditto  | |
| llvm/lib/TargetParser/X86TargetParser.cpp | ||
| 214 | Unnecessary line break  | |
| 659 | Should this be AVX2 like all the other integer features?  | |
| llvm/lib/Target/X86/X86.td | ||
|---|---|---|
| 243 | ISE says it only requires AVX  | |
LGTM - but I'd prefer more complete 32-bit vs 64-bit test coverage (similar to the SM3/SM4 patches) if its possible.
alphabetical order