[PATCH] D142501: [clang][deps] Fix modulemap file path for implementation of module

Ben Langmuir via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 24 13:07:59 PST 2023


benlangmuir marked an inline comment as done.
benlangmuir added inline comments.


================
Comment at: clang/test/ClangScanDeps/modules-implementation-vfs.m:67-68
+#import <B/B.h>
+struct A { float x; }; // expected-error {{incompatible definitions}} expected-note {{type 'float' here}}
+// expected-note at frameworks/A.framework/Headers/A.h:1 {{type 'int' here}}
+
----------------
jansvoboda11 wrote:
> What purpose does this check serve?
Added a comment to explain this check.


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

https://reviews.llvm.org/D142501



More information about the cfe-commits mailing list