[llvm-commits] PATCH: correct ARM NOP encoding

James Molloy James.Molloy at arm.com
Tue Dec 4 10:57:38 PST 2012


Hi,

LGTM, the original is obviously wrong.

Cheers,

James
________________________________________
From: llvm-commits-bounces at cs.uiuc.edu [llvm-commits-bounces at cs.uiuc.edu] On Behalf Of David Sehr [sehr at google.com]
Sent: 04 December 2012 18:31
To: llvm-commits at cs.uiuc.edu
Subject: [llvm-commits] PATCH: correct ARM NOP encoding

The encoding of NOP in ARMAsmBackend.cpp is missing a trailing zero, which causes the emission of a coprocessor instruction rather than "mov r0, r0" as indicated in the comment.
The test also checks for the wrong encoding.

Review and comments are now requested.

David


-- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium.  Thank you.





More information about the llvm-commits mailing list