FYI, duplicate includes in rustllvm.h #52105

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.

Sign up for GitHub

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

Jump to bottom

Closed

wqweto opened this issue Jul 6, 2018 · 0 comments · Fixed by #52107

wqweto commented Jul 6, 2018

In rustllvm.h on line 24 there are these harmless duplicate includes, not sure if these are left on purpose

#include "llvm/IR/InlineAsm.h"
#include "llvm/IR/InlineAsm.h"
#include "llvm/IR/LLVMContext.h"
#include "llvm/IR/LLVMContext.h"

(Easier for me to report here than to review/sign project contributing guidelines/agreement as I'm not planning on contributing per se)

The text was updated successfully, but these errors were encountered:

PramodBisht mentioned this issue

removed redundant header file import in rustllvm.h #52107

Merged

Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this issue


          Rollup merge of rust-lang#52107 - PramodBisht:feature/52105, r=rkruppe

f532daa

removed redundant header file import in rustllvm.h

fix rust-lang#52105
removed redundant header file import in rustllvm.h
cc @wqweto

bors closed this as completed in #52107

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