Skip to content

[BUG] Regression output dpends on temporary directory #1225

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
MaxSagebaum opened this issue Aug 14, 2024 · 2 comments
Open

[BUG] Regression output dpends on temporary directory #1225

MaxSagebaum opened this issue Aug 14, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@MaxSagebaum
Copy link
Contributor

In the regression test run https://github.com/hsutter/cppfront/actions/runs/10382592222/job/28745930035?pr=1224. I got a diff for an linking of main in a random directory. We probably should remove the user of temporary intermediate targets.

diff --git a/regression-tests/test-results/gcc-10-c++20/mixed-type-safety-1.cpp.output b/regression-tests/test-results/gcc-10-c++20/mixed-type-safety-1.cpp.output
index 06f8418..bc55601 100644
--- a/regression-tests/test-results/gcc-10-c++20/mixed-type-safety-1.cpp.output
+++ b/regression-tests/test-results/gcc-10-c++20/mixed-type-safety-1.cpp.output
@@ -1,4 +1,4 @@
-/usr/bin/ld: /tmp/ccSlwpC6.o: in function `main':
+/usr/bin/ld: /tmp/ccJ7QUsS.o: in function `main':
 mixed-type-safety-1.cpp:(.text+0x327): undefined reference to `void print<std::integral_constant<bool, false> >(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::integral_constant<bool, false> const&)'
 /usr/bin/ld: mixed-type-safety-1.cpp:(.text+0x388): undefined reference to `void print<std::integral_constant<bool, true> >(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::integral_constant<bool, true> const&)'
 /usr/bin/ld: mixed-type-safety-1.cpp:(.text+0x422): undefined reference to `void print<std::integral_constant<bool, true> >(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::integral_constant<bool, true> const&)'
@filipsajdak
Copy link
Contributor

I have tried to use the -pipe option, but gcc is still using the temporary file.

Nevertheless, this output identified an issue that should be addressed rather than be updated - more here: #1203 (comment)

@MaxSagebaum
Copy link
Contributor Author

In my experience gcc uses the temporary directory when compiling from source to an executable. When using intermediate objects *.o files, now temporary files should be generated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants