tahonermann added a comment.

I don't know the driver code very well, but as best I can tell, this appears to 
match the design agreed to at the last Clang Modules WG meeting.



================
Comment at: clang/test/Driver/module-output.cppm:5
+//
+// Tests that the .pcm file will be generated in the same direcotry with the 
specified
+// output and the name of the .pcm file should be the same with the input file.
----------------



================
Comment at: clang/test/Driver/module-output.cppm:18
+// RUN: %clang %t/Hello.cppm -fmodule-output -arch i386 -arch x86_64 -### 
-target \
+// RUN:   x86_64-apple-darwin 2>&1 | FileCheck %t/Hello.cppm 
-check-prefix=MULTIPLR-ARCH
+
----------------



================
Comment at: clang/test/Driver/module-output.cppm:26
+
+// MULTIPLR-ARCH: option '-fmodule-output' can't be used with multiple arch 
options
+
----------------



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

https://reviews.llvm.org/D137058

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to