[PATCH] D73227: [ms] [llvm-ml] Use default RIP-relative addressing for x64 MASM.

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo added a comment.

In D73227#1891896 <https://reviews.llvm.org/D73227#1891896>, @epastor wrote:

> In D73227#1891874 <https://reviews.llvm.org/D73227#1891874>, @thakis wrote:
>
> > What's the plan for 64-bit inline asm?
>
>
> It should probably be the same, admittedly. To clarify: isParsingInlineAsm will only be true when parsing inline MS-style assembly? If so, I'll rename it to isParsingInlineMSAsm in this patch, and let the handling trigger off of both.


Are there any cases where MS style assembly can be injected inline on x86_64? As far as I know, MSVC didn't support any inline assembly at all on platforms other than i386.

If that's the case, I think the sane thing would be to keep handling inline asm like it always has been, as gcc style inline assembly. There could, concievably, be projects that contains gcc style inline assembly within `#ifdef __clang__` (as this probably worked so far).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73227/new/

https://reviews.llvm.org/D73227