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

llvm-config with src!=obj omits -I<obj_dir>/include #4853

Closed
llvmbot opened this issue Jun 29, 2009 · 1 comment
Closed

llvm-config with src!=obj omits -I<obj_dir>/include #4853

llvmbot opened this issue Jun 29, 2009 · 1 comment
Labels
bugzilla Issues migrated from bugzilla llvm-tools All llvm tools that do not have corresponding tag

Comments

@llvmbot
Copy link
Collaborator

llvmbot commented Jun 29, 2009

Bugzilla Link 4481
Resolution FIXED
Resolved on Jul 07, 2009 17:17
Version trunk
OS Linux
Attachments Proposed fix
Reporter LLVM Bugzilla Contributor
CC @asl,@lattner

Extended Description

jyasskin@enki:~/opensource/llvm/oprof/dbg$ cat test_config.cc
#include "llvm/Module.h"

int main() {
}

jyasskin@enki:~/opensource/llvm/oprof/dbg$ ./Debug/bin/llvm-config --cxxflags
-I/usr/local/google/jyasskin/llvm/oprof/dbg/../src/include -D_DEBUG -D_GNU_SOURCE -D__STDC_LIMIT_MACROS -D__STDC_CONSTANT_MACROS -g -fPIC -Woverloaded-virtual

jyasskin@enki:/opensource/llvm/oprof/dbg$ g++ -c test_config.cc ./Debug/bin/llvm-config --cxxflags 2>&1|head -20
In file included from /usr/local/google/jyasskin/llvm/oprof/dbg/../src/include/llvm/Value.h:18,
from /usr/local/google/jyasskin/llvm/oprof/dbg/../src/include/llvm/User.h:22,
from /usr/local/google/jyasskin/llvm/oprof/dbg/../src/include/llvm/Constant.h:17,
from /usr/local/google/jyasskin/llvm/oprof/dbg/../src/include/llvm/GlobalValue.h:21,
from /usr/local/google/jyasskin/llvm/oprof/dbg/../src/include/llvm/Function.h:21,
from /usr/local/google/jyasskin/llvm/oprof/dbg/../src/include/llvm/Module.h:18,
from test_config.cc:1:
/usr/local/google/jyasskin/llvm/oprof/dbg/../src/include/llvm/Use.h:29:31: error: llvm/ADT/iterator.h: No such file or directory
...
jyasskin@enki:
/opensource/llvm/oprof/dbg$ ls include/llvm/ADT/iterator.h
include/llvm/ADT/iterator.h

If llvm-config --cxxflags included "-I/usr/local/google/jyasskin/llvm/oprof/dbg/include", this would compile successfully.

@llvmbot
Copy link
Collaborator Author

llvmbot commented Jul 8, 2009

Fixed by r74956.

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 3, 2021
cyndyishida pushed a commit to cyndyishida/llvm-project that referenced this issue Jun 29, 2022
…lvm#4853)

Instead, just pop the cleanups at the end of the asm statement.

This fixes an assertion failure in BuildStmtExpr. It also fixes a bug where blocks and C compound literals were destructed at the end of the asm statement instead of at the end of the enclosing scope.

Differential Revision: https://reviews.llvm.org/D125936

(cherry picked from commit 8fc3d71)
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 llvm-tools All llvm tools that do not have corresponding tag
Projects
None yet
Development

No branches or pull requests

1 participant