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

API Header/Implementation mismatch #59244

Closed
Naville opened this issue Nov 29, 2022 · 3 comments
Closed

API Header/Implementation mismatch #59244

Naville opened this issue Nov 29, 2022 · 3 comments

Comments

@Naville
Copy link
Contributor

Naville commented Nov 29, 2022

llvm/IR/Verifier.h

llvm/IR/Verifier.cpp

@Naville
Copy link
Contributor Author

Naville commented Nov 29, 2022

In the header we have:

bool verifyModule(const Module &M, raw_ostream *OS = nullptr,
                  bool *BrokenDebugInfo = nullptr);
bool verifyModule(bool &BrokenDebugInfo, const Module &M, raw_ostream *OS);

But only the former is actually implemented in the cpp, resulting in:

Undefined symbols for architecture arm64:
  "llvm::verifyModule(bool&, llvm::Module const&, llvm::raw_ostream*)", referenced from:

@Naville
Copy link
Contributor Author

Naville commented Nov 29, 2022

Seems at least a issue since LLVM 11, didn't bother to look furthur

@Naville
Copy link
Contributor Author

Naville commented Nov 29, 2022

Date back to e365618 where the declaration is added without implementation?

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

2 participants