================
@@ -0,0 +1,13 @@
+// This test checks if Window PE file compiled with -flto option contains a 
magic 
+// string "LTCG" to indicate LTO compilation.
+
+// REQUIRES: system-windows
+
+// RUN: %clang --target=x86_64-pc-windows-msvc -flto -fuse-ld=lld %s -o %t.exe
+// RUN: dumpbin /HEADERS %t.exe | FileCheck %s
----------------
mikolaj-pirog wrote:

> @mikolaj-pirog One of the goals of LLVM is cross compilation. `dumpbin` will 
> not be available on Linux. We usually use `REQUIRES: system-windows` for 
> testing low-level stuff on that system. For this case you’d probably want to 
> change `llvm-objdump` to match `dumpbin` behavior.
Okay, I think I will do the following: I will change the tests to use 
`llvm-objdump` with `-s` option to simply check for presence of the  magic 
strings, for now accepting the problems mentioned above. After this patch is 
merged, I will prepare new patch introducing an option to `llvm-objdump` to 
display contents of debug dir, the way `dumpbin` does, and correct the tests. 
Would that be OK?

https://github.com/llvm/llvm-project/pull/114260
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to