[LLVMdev] x86 Intel Syntax and MASM 9.x

[Eli Friedman]

On Tue, Jun 16, 2009 at 9:39 AM, Gaster,
Benedict<Benedict.Gaster at amd.com> wrote:
> Hi Eli,
>
> Yep I was being stupid.
>
> Please find attached a patch for initial changes to get MASM working.

Patch looks fine except that it has tabs (LLVM uses only spaces for
indentation).  Also, can you generate the patch using "svn diff"?
It's currently in some unusual format which "patch" doesn't recognize.

> There is still one problem that I am looking into around changing
> alignments within SEGMENTS. The problem is that MASM allows 2,4,16,256
> alignments, default being 16, but LLVM is sometimes generating 32
> alignment, for example, consider the following code:

Huh.  For correctness, I guess you'll have to round up the alignment,
and abort on anything higher than 256.  (Also, perhaps the front-end
should avoid generating such constructs in the first place when
targeting Windows, but that's a separate issue.)

-Eli