[PATCH] Simplify the logic in getX86SubSuperRegister

Eli Bendersky eliben at google.com
Fri Feb 15 14:02:27 PST 2013


Hello,

getX86SubSuperRegister has a special mode with High=true for i64 which
exists solely to enable it to call itself for i8 with some registers.
The proposed patch simplifies the function somewhat to make the High
bit only meaningful for the i8 mode, which makes sense. No functional
difference (getX86SubSuperRegister is not getting called from anywhere
outside with i64 and High=true).

If this looks OK, I'll also add a comment that documents
getX86SubSuperRegister propertly.

Eli
-------------- next part --------------
A non-text attachment was scrubbed...
Name: simplify_getx86subsuperregister.1.patch
Type: application/octet-stream
Size: 1935 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130215/3f54ec89/attachment.obj>


More information about the llvm-commits mailing list