[PATCH] D20385: [x86] avoid printing unnecessary sign bits of hex immediates in asm comments (PR20347)

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Fri May 27 07:08:16 PDT 2016


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

In http://reviews.llvm.org/D20385#442168, @spatel wrote:

> In http://reviews.llvm.org/D20385#442104, @RKSimon wrote:
>
> > Do we have a similar issue with Intel syntax?
>
>
> We would, except that we don't print any comments at all for Intel syntax. :)
>  I can make that a TODO / follow-up.


OK - please can you raise a bugzilla about improving intel assembly comments (and reference this patch)?

LGTM.


http://reviews.llvm.org/D20385





More information about the llvm-commits mailing list