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

Section printed into .ll file #1457

Closed
asl opened this issue Jan 6, 2007 · 10 comments
Closed

Section printed into .ll file #1457

asl opened this issue Jan 6, 2007 · 10 comments
Assignees
Labels
bugzilla Issues migrated from bugzilla compile-fail Use [accepts-invalid] and [rejects-valid] instead llvm-tools All llvm tools that do not have corresponding tag

Comments

@asl
Copy link
Collaborator

asl commented Jan 6, 2007

Bugzilla Link 1085
Resolution FIXED
Resolved on Nov 07, 2018 00:17
Version trunk
OS Linux
Blocks #1439
Attachments Preprocessed source, Resulted .ll, Slightly reduced source

Extended Description

Attached source (from Qt) will cause sections of form "gnu.linkonce.t" to be
printed into resulting LLVM bytecode (e.g. when compiling with -emit-llvm -S).

This causes misc linker problems.

A quick workaround was introduced:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20070101/042142.html

It should be reverted after this bug fixed.

@asl
Copy link
Collaborator Author

asl commented Jan 6, 2007

assigned to @lattner

@lattner
Copy link
Collaborator

lattner commented Jan 7, 2007

This is a ELF-specific bug in the C++ FE. Patch sent to llvm-commits, testcase here:
Regression/C++Frontend/2007-01-06-ELF-Thunk-Sections.cpp

-Chris

@asl
Copy link
Collaborator Author

asl commented Jan 7, 2007

Workaround was removed as well.

@lattner
Copy link
Collaborator

lattner commented Jan 7, 2007

@llvmbot
Copy link
Collaborator

llvmbot commented Jan 8, 2007

This is not fixed. The Regression/C++Frontend/2007-01-06-ELF-Thunk-Sections.cpp
test case fails on Linux at least. The "not grep" part of it is failing.

@asl
Copy link
Collaborator Author

asl commented Jan 8, 2007

Hmm. Strange. I've got linker errors due to presense of such sections. Now:

  1. Sections go away
  2. There are no linker errors.

Will check the testcase.

@asl
Copy link
Collaborator Author

asl commented Jan 8, 2007

Reid, could you please attach the LLVM bytecode generated by llvm-g++ from the
testcase?

@llvmbot
Copy link
Collaborator

llvmbot commented Jan 8, 2007

x86-linux bytecode for test case
This bytecode file provides the output of llvm-gcc for x86-linux where the test
case fails.

@lattner
Copy link
Collaborator

lattner commented Jan 8, 2007

This fixes the fix. Reid, plz verify and remove the xfail from the testcase if ok now.

http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20070108/042246.html

Thanks,

-Chris

@llvmbot
Copy link
Collaborator

llvmbot commented Jan 9, 2007

Works now.

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 3, 2021
kitano-metro pushed a commit to RIKEN-RCCS/llvm-project that referenced this issue Feb 14, 2023
kitano-metro pushed a commit to RIKEN-RCCS/llvm-project that referenced this issue Feb 14, 2023
kitano-metro pushed a commit to RIKEN-RCCS/llvm-project that referenced this issue Feb 14, 2023
kitano-metro pushed a commit to RIKEN-RCCS/llvm-project that referenced this issue Feb 14, 2023
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla compile-fail Use [accepts-invalid] and [rejects-valid] instead llvm-tools All llvm tools that do not have corresponding tag
Projects
None yet
Development

No branches or pull requests

3 participants