Skip to content
This repository has been archived by the owner on Jul 1, 2023. It is now read-only.

Lit tests fail with rebased branch. #148

Open
Sameeranjoshi opened this issue May 5, 2021 · 4 comments
Open

Lit tests fail with rebased branch. #148

Sameeranjoshi opened this issue May 5, 2021 · 4 comments

Comments

@Sameeranjoshi
Copy link
Contributor

Sameeranjoshi commented May 5, 2021

While I was trying to run
ninja check-bolt on the rebased branch following tests fail.


********************
********************
Failed Tests (17):
  BOLT :: X86/debugTypesBug.s
  BOLT :: X86/fix-branches-jrcxz.s
  BOLT :: X86/hot_end_symbol.s
  BOLT :: X86/instrumentation-dup-jts.s
  BOLT :: X86/instrumentation-ind-calls.s
  BOLT :: X86/interprocedural_ref_entry_point.s
  BOLT :: X86/issue20.s
  BOLT :: X86/no-output.test
  BOLT :: X86/pre-aggregated-perf.test
  BOLT :: X86/section_reloc_with_addend.s
  BOLT :: X86/shrinkwrapping-alignment.s
  BOLT :: X86/shrinkwrapping-critedge.s
  BOLT :: X86/shrinkwrapping-insertcfi.s
  BOLT :: X86/shrinkwrapping-lock.s
  BOLT :: X86/shrinkwrapping-pushpop.s
  BOLT :: X86/user-func-reorder.c
  BOLT :: X86/zero-sized-object.s


Testing Time: 0.52s
  Passed:  9
  Failed: 17

I am on an AMD Ryzen 7 hardware

Architecture:        x86_64
CPU op-mode(s):      32-bit, 64-bit
Byte Order:          Little Endian

I see one frequent issue with many files.

hot_end_symbol.s.tmp.o: relocation R_X86_64_32S against `.data' can not be used when making a PIE object; recompile with -fPIC
/usr/bin/ld: final link failed: Nonrepresentable section on output

Are there some suggestions on fixing them maybe I can try sending a patch if that's trivial?

@maksfb
Copy link
Contributor

maksfb commented May 5, 2021

Could you check if the compiler on your system produces position-independent executables (PIEs) by default?

@yota9
Copy link
Contributor

yota9 commented May 5, 2021

@maksfb
Yes, I have the same problem. Since gcc/ld on the latest ubuntu produces PIE binaries by default I've passed %cflags with -no-pie option to all the tests to be able to test my changes. If you'd like I could make pull request with this changes.

@maksfb
Copy link
Contributor

maksfb commented May 5, 2021

@yota9 - thanks, yes this will be great!

@aaupov
Copy link
Contributor

aaupov commented May 10, 2021

@maksfb: we have these tests failing with the latest ubuntu due to pie (turned on by default since 17.10: https://wiki.ubuntu.com/Security/Features, Built as PIE)

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

No branches or pull requests

4 participants