Skip to content
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

32-bit tests broken in trunk #308

Closed
ramosian-glider opened this issue Sep 1, 2015 · 4 comments
Closed

32-bit tests broken in trunk #308

ramosian-glider opened this issue Sep 1, 2015 · 4 comments

Comments

@ramosian-glider
Copy link
Member

Originally reported on Google Code with ID 308

What steps will reproduce the problem?
1. build Clang on x86_64
2. run `make -j10 check-asan'
3. run `file projects/compiler-rt/test/asan/32bitConfig/TestCases/Output/printf-1.c.tmp'

What is the expected output?
  ELF 32-bit LSB executable

What do you see instead?
  ELF 64-bit LSB executable

Looks like the error was introduced in r207707. Here are the issues that I see with
this revision:
* target_cflags got moved to test/lit.site.cfg.in from test/asan/lit.site.cfg.in; but
COMPILER_RT_TEST_COMPILER_CFLAGS is still set in test/asan/CMakeLists.txt so it's not
available when configuring test/lit.common.configure.in
* test/lit.common.configured.in uses COMPILER_RT_TEST_COMPILER_FLAGS instead of COMPILER_RT_TEST_COMPILER_CFLAGS

Reported by tetra2005 on 2014-05-07 15:07:37

@ramosian-glider
Copy link
Member Author

Thanks for the detailed report.  Can you please confirm r207707 fixes the issues?

Reported by garious on 2014-05-07 19:32:13

@ramosian-glider
Copy link
Member Author

sorry, fixed in r208240

Reported by garious on 2014-05-07 19:34:39

@ramosian-glider
Copy link
Member Author

Seems to be working. Thanks!

Reported by tetra2005 on 2014-05-08 11:56:31

  • Status changed: Fixed

@ramosian-glider
Copy link
Member Author

Adding Project:AddressSanitizer as part of GitHub migration.

Reported by ramosian.glider on 2015-07-30 09:14:08

  • Labels added: ProjectAddressSanitizer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant