[PATCH] D37466: D37461: fixups for existing InlineAsm tests + adding new ones
Reid Kleckner via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 2 09:31:49 PDT 2017
rnk added inline comments.
================
Comment at: test/CodeGen/ms-inline-asm.cpp:37-38
- int lvar = 10;
- __asm mov eax, offset Foo::ptr
- __asm mov eax, offset Foo::Bar::ptr
-// CHECK-LABEL: define void @_Z2t2v()
----------------
coby wrote:
> rnk wrote:
> > These don't seem tested anywhere now
> I've tested them against msvc, and they are seem to be unsupported there as well, so I don't see any value in keeping this one around :\
Got it. Can we make a negative test for offset with namespace separators then?
Repository:
rL LLVM
https://reviews.llvm.org/D37466
More information about the cfe-commits
mailing list