<div dir="ltr">On reflection itanium wouldn't be sufficient/my comment wasn't sufficiently descriptive (I realized after I made this change that it would also fix the MacOS buildbot failure I was seeing & hadn't understood) - Darwin doesn't use comdats, for example, but is still an itanium ABI.<br><br>I could remove the checks for comdats - but I think they're beneficial/relevant.<br><br>Could refactor them to be optional somehow, but not sure it's worth it?</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jan 30, 2017 at 9:28 AM Robinson, Paul <<a href="mailto:paul.robinson@sony.com">paul.robinson@sony.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Use %itanium_abi_triple instead?<br class="gmail_msg">
<br class="gmail_msg">
> -----Original Message-----<br class="gmail_msg">
> From: cfe-commits [mailto:<a href="mailto:cfe-commits-bounces@lists.llvm.org" class="gmail_msg" target="_blank">cfe-commits-bounces@lists.llvm.org</a>] On Behalf Of<br class="gmail_msg">
> David Blaikie via cfe-commits<br class="gmail_msg">
> Sent: Sunday, January 29, 2017 9:34 PM<br class="gmail_msg">
> To: <a href="mailto:cfe-commits@lists.llvm.org" class="gmail_msg" target="_blank">cfe-commits@lists.llvm.org</a><br class="gmail_msg">
> Subject: r293457 - Tidy up codegen modules test & make it x86 specific<br class="gmail_msg">
> since it relies on Itanium name manglings<br class="gmail_msg">
><br class="gmail_msg">
> Author: dblaikie<br class="gmail_msg">
> Date: Sun Jan 29 23:33:51 2017<br class="gmail_msg">
> New Revision: 293457<br class="gmail_msg">
><br class="gmail_msg">
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=293457&view=rev" rel="noreferrer" class="gmail_msg" target="_blank">http://llvm.org/viewvc/llvm-project?rev=293457&view=rev</a><br class="gmail_msg">
> Log:<br class="gmail_msg">
> Tidy up codegen modules test & make it x86 specific since it relies on<br class="gmail_msg">
> Itanium name manglings<br class="gmail_msg">
><br class="gmail_msg">
> Modified:<br class="gmail_msg">
>     cfe/trunk/test/Modules/codegen.test<br class="gmail_msg">
><br class="gmail_msg">
> Modified: cfe/trunk/test/Modules/codegen.test<br class="gmail_msg">
> URL: <a href="http://llvm.org/viewvc/llvm-" rel="noreferrer" class="gmail_msg" target="_blank">http://llvm.org/viewvc/llvm-</a><br class="gmail_msg">
> project/cfe/trunk/test/Modules/codegen.test?rev=293457&r1=293456&r2=293457<br class="gmail_msg">
> &view=diff<br class="gmail_msg">
> ==========================================================================<br class="gmail_msg">
> ====<br class="gmail_msg">
> --- cfe/trunk/test/Modules/codegen.test (original)<br class="gmail_msg">
> +++ cfe/trunk/test/Modules/codegen.test Sun Jan 29 23:33:51 2017<br class="gmail_msg">
> @@ -1,15 +1,16 @@<br class="gmail_msg">
>  RUN: rm -rf %t<br class="gmail_msg">
> +REQUIRES: x86-registered-target<br class="gmail_msg">
><br class="gmail_msg">
> -RUN: %clang_cc1 -fmodules-codegen -x c++ -fmodules -emit-module -fmodule-<br class="gmail_msg">
> name=foo %S/Inputs/codegen/foo.modulemap -o %t/foo.pcm<br class="gmail_msg">
> -RUN: %clang_cc1 -fmodules-codegen -x c++ -fmodules -emit-module -fmodule-<br class="gmail_msg">
> name=bar %S/Inputs/codegen/bar.modulemap -o %t/bar.pcm -fmodule-<br class="gmail_msg">
> file=%t/foo.pcm<br class="gmail_msg">
> +RUN: %clang_cc1 -triple=x86_64-linux-gnu -fmodules-codegen -x c++ -<br class="gmail_msg">
> fmodules -emit-module -fmodule-name=foo %S/Inputs/codegen/foo.modulemap -o<br class="gmail_msg">
> %t/foo.pcm<br class="gmail_msg">
> +RUN: %clang_cc1 -triple=x86_64-linux-gnu -fmodules-codegen -x c++ -<br class="gmail_msg">
> fmodules -emit-module -fmodule-name=bar %S/Inputs/codegen/bar.modulemap -o<br class="gmail_msg">
> %t/bar.pcm -fmodule-file=%t/foo.pcm<br class="gmail_msg">
><br class="gmail_msg">
> -RUN: %clang_cc1 -emit-llvm %t/foo.pcm -o - | FileCheck --check-prefix=FOO<br class="gmail_msg">
> %s<br class="gmail_msg">
> -RUN: %clang_cc1 -emit-llvm %t/bar.pcm -o - -fmodule-file=%t/foo.pcm |<br class="gmail_msg">
> FileCheck --check-prefix=BAR-CMN --check-prefix=BAR %s<br class="gmail_msg">
> -RUN: %clang_cc1 -fmodules -fmodule-file=%t/foo.pcm -fmodule-<br class="gmail_msg">
> file=%t/bar.pcm %S/Inputs/codegen/use.cpp -emit-llvm -o - | FileCheck --<br class="gmail_msg">
> check-prefix=USE-CMN --check-prefix=USE %s<br class="gmail_msg">
> +RUN: %clang_cc1 -triple x86_64-linux-gnu -emit-llvm -o - %t/foo.pcm |<br class="gmail_msg">
> FileCheck --check-prefix=FOO %s<br class="gmail_msg">
> +RUN: %clang_cc1 -triple x86_64-linux-gnu -emit-llvm -o - %t/bar.pcm -<br class="gmail_msg">
> fmodule-file=%t/foo.pcm | FileCheck --check-prefix=BAR-CMN --check-<br class="gmail_msg">
> prefix=BAR %s<br class="gmail_msg">
> +RUN: %clang_cc1 -triple x86_64-linux-gnu -emit-llvm -o - -fmodules -<br class="gmail_msg">
> fmodule-file=%t/foo.pcm -fmodule-file=%t/bar.pcm %S/Inputs/codegen/use.cpp<br class="gmail_msg">
> | FileCheck --check-prefix=USE-CMN --check-prefix=USE %s<br class="gmail_msg">
><br class="gmail_msg">
> -RUN: %clang_cc1 -O2 -disable-llvm-passes -emit-llvm %t/foo.pcm -o - |<br class="gmail_msg">
> FileCheck --check-prefix=FOO %s<br class="gmail_msg">
> -RUN: %clang_cc1 -O2 -disable-llvm-passes -emit-llvm %t/bar.pcm -o - -<br class="gmail_msg">
> fmodule-file=%t/foo.pcm | FileCheck --check-prefix=BAR-CMN --check-<br class="gmail_msg">
> prefix=BAR-OPT %s<br class="gmail_msg">
> -RUN: %clang_cc1 -O2 -disable-llvm-passes -fmodules -fmodule-<br class="gmail_msg">
> file=%t/foo.pcm -fmodule-file=%t/bar.pcm %S/Inputs/codegen/use.cpp -emit-<br class="gmail_msg">
> llvm -o - | FileCheck --check-prefix=USE-CMN --check-prefix=USE-OPT %s<br class="gmail_msg">
> +RUN: %clang_cc1 -triple x86_64-linux-gnu -emit-llvm -o - -O2 -disable-<br class="gmail_msg">
> llvm-passes %t/foo.pcm | FileCheck --check-prefix=FOO %s<br class="gmail_msg">
> +RUN: %clang_cc1 -triple x86_64-linux-gnu -emit-llvm -o - -O2 -disable-<br class="gmail_msg">
> llvm-passes %t/bar.pcm -fmodule-file=%t/foo.pcm | FileCheck --check-<br class="gmail_msg">
> prefix=BAR-CMN --check-prefix=BAR-OPT %s<br class="gmail_msg">
> +RUN: %clang_cc1 -triple x86_64-linux-gnu -emit-llvm -o - -O2 -disable-<br class="gmail_msg">
> llvm-passes -fmodules -fmodule-file=%t/foo.pcm -fmodule-file=%t/bar.pcm<br class="gmail_msg">
> %S/Inputs/codegen/use.cpp | FileCheck --check-prefix=USE-CMN --check-<br class="gmail_msg">
> prefix=USE-OPT %s<br class="gmail_msg">
><br class="gmail_msg">
>  FOO-NOT: comdat<br class="gmail_msg">
>  FOO: $_Z3foov = comdat any<br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> _______________________________________________<br class="gmail_msg">
> cfe-commits mailing list<br class="gmail_msg">
> <a href="mailto:cfe-commits@lists.llvm.org" class="gmail_msg" target="_blank">cfe-commits@lists.llvm.org</a><br class="gmail_msg">
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" class="gmail_msg" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br class="gmail_msg">
</blockquote></div>