This is an archive of the discontinued LLVM Phabricator instance.

[ARM] Do not convert some vmov instructions
ClosedPublic

Authored by miyuki on Mar 20 2018, 4:19 AM.

Details

Summary

Patch https://reviews.llvm.org/D44467 implements conversion of invalid
vmov instructions into valid ones. It turned out that some valid
instructions also get converted, for example

vmov.i64 d2, #0xff00ff00ff00ff00 ->
vmov.i16 d2, #0xff00

Such behavior is incorrect because according to the ARM ARM section
F2.7.7 Modified immediate constants in T32 and A32 Advanced SIMD
instructions, "On assembly, the data type must be matched in the table
if possible."

This patch fixes the isNEONmovReplicate check so that the above
instruction is not modified any more.

Diff Detail

Repository
rL LLVM

Event Timeline

miyuki created this revision.Mar 20 2018, 4:19 AM
rengolin accepted this revision.Apr 3 2018, 1:05 PM
This revision is now accepted and ready to land.Apr 3 2018, 1:05 PM
This revision was automatically updated to reflect the committed changes.