<div dir="ltr"><div>+# RUN: modularize %s -x c++ 2>&1 | FileCheck %s<br></div><div><br></div><div style>2>&1 will confuse the testing infrastructure, which doesn't accept full bash/sh syntax. See near the bottom of this section:</div>
<div><a href="http://llvm.org/docs/TestingGuide.html#writing-new-regression-tests">http://llvm.org/docs/TestingGuide.html#writing-new-regression-tests</a><br></div><div><br></div><div>+// SomeTypes.h - Define a few different kinds of types - no modules problems.</div>
<div><br></div><div style>You can probably drop the filename. Over time it's unlikely that all of the added tests will have the filename, so things will gradually become inconsistent.</div><div><br></div><div>+</div><div>
+typedef int TypeInt;</div><div>+</div><div>+typedef TypeInt NestedTypeInt;</div><div>+</div><div>+struct TypeStruct</div><div>+{</div><div>+  int Member;</div><div>+};</div><div><br></div><div style>Keep the opening curly bracket on the same line as "struct TypeStruct" as per the LLVM coding conventions. For simplicity, you may want to just run clang-format on the files.</div>
<div style><br></div><div style>+# CHECK: error: 'SYMBOL' defined at both {{.*}}{{\\|/}}InputProblemsInconsistent{{\\|/}}SubHeader.h:3:9 and {{.*}}{{\\|/}}InputProblemsInconsistent/SubHeader.h:6:9<br></div><div style>
<br></div><div style>It seems like other test in tree do {{[/\\]}} instead of {{\\|/}}. Rather microscopic, but it would be good to be consistent.</div><div style><br></div><div style>-- Sean Silva</div></div>