[PATCH] D137058: [Driver] [Modules] Support -fmodule-output (1/2)

[Tom Honermann via Phabricator via cfe-commits]

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