This is an archive of the discontinued LLVM Phabricator instance.

Add check for tied operands
ClosedPublic

Authored by sidneym on Aug 10 2018, 4:59 PM.

Details

Summary

The "tied" attribute was not being checked.
When the registers mismatched no error is generated unless asserts are enabled.

Diff Detail

Repository
rL LLVM

Event Timeline

sidneym created this revision.Aug 10 2018, 4:59 PM
kparzysz accepted this revision.Aug 13 2018, 5:34 AM

In the longer term we should probably make the error messages more self-explanatory.

This revision is now accepted and ready to land.Aug 13 2018, 5:34 AM
This revision was automatically updated to reflect the committed changes.
bcain added inline comments.Aug 13 2018, 7:31 AM
llvm/trunk/test/MC/Hexagon/tied-ops.s
1

I think you should use this run line instead:

# RUN: not llvm-mc -arch=hexagon -filetype=asm %s 2>%t; FileCheck --implicit-check-not="error:" %s <%t