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

[NFC] Remove dead code #75336

Merged
merged 1 commit into from
Dec 13, 2023
Merged

Conversation

abhina-sree
Copy link
Contributor

Remove dead code

@abhina-sree abhina-sree merged commit c9e1003 into llvm:main Dec 13, 2023
4 of 5 checks passed
@abhina-sree abhina-sree deleted the abhina/remove_dead branch December 13, 2023 13:05
@llvmbot
Copy link
Collaborator

llvmbot commented Dec 13, 2023

@llvm/pr-subscribers-llvm-support

Author: Abhina Sree (abhina-sree)

Changes

Remove dead code


Full diff: https://github.com/llvm/llvm-project/pull/75336.diff

1 Files Affected:

  • (modified) llvm/include/llvm/Support/AutoConvert.h (-1)
diff --git a/llvm/include/llvm/Support/AutoConvert.h b/llvm/include/llvm/Support/AutoConvert.h
index 6608dd461d7265..79566189d81976 100644
--- a/llvm/include/llvm/Support/AutoConvert.h
+++ b/llvm/include/llvm/Support/AutoConvert.h
@@ -30,7 +30,6 @@ extern "C" {
 int enableAutoConversion(int FD);
 int disableAutoConversion(int FD);
 int restoreStdHandleAutoConversion(int FD);
-int overrideAutoConversion(int FD, char *Filetag);
 #ifdef __cplusplus
 }
 #endif // __cplusplus

@dwblaikie
Copy link
Collaborator

Once sent for review, please don't commit changes until the review is completed.

If the change does not need pre-commit review (per the LLVM developers policy), please don't send a pull request, instead commit it directly.

@abhina-sree
Copy link
Contributor Author

Once sent for review, please don't commit changes until the review is completed.

If the change does not need pre-commit review (per the LLVM developers policy), please don't send a pull request, instead commit it directly.

Thanks for letting me know, I will make sure to do this next time

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

Successfully merging this pull request may close these issues.

None yet

3 participants