benhamilton accepted this revision.
benhamilton added a comment.
This revision is now accepted and ready to land.
In the diff description, please fix the typo: `Do not threat the asm clobber`
-> `Do not treat the asm clobber`
================
Comment at: unittests/Format/FormatTest.cpp:12756-12763
+TEST_F(FormatTest, GuessedLanguageWithInlineAsmClobbers) {
+ EXPECT_EQ(FormatStyle::LK_Cpp, guessLanguage("foo.h",
+ "void f() {\n"
+ " asm (\"mov %[e], %[d]\"\n"
+ " : [d] \"=rm\" (d)\n"
+ " [e] \"rm\" (*e));\n"
+ "}"));
----------------
Worth a second test with `asm volatile (...)` ?
Repository:
rC Clang
https://reviews.llvm.org/D54111
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
