[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