https://github.com/abhina-sree created https://github.com/llvm/llvm-project/pull/146680
This patch adds back the needed AutoConvert.h header and removes the unneeded include guard of MVS to prevent this header from being removed in the future >From 1b0722ba6f3c8c580f31f4dd307fedd85c8383c2 Mon Sep 17 00:00:00 2001 From: Abhina Sreeskantharajan <abhina.sreeskanthara...@ibm.com> Date: Wed, 2 Jul 2025 08:33:44 -0400 Subject: [PATCH] remove helper function and use target agnostic needConversion function --- clang/lib/Basic/SourceManager.cpp | 17 ++++------------- 1 file changed, 4 insertions(+), 13 deletions(-) diff --git a/clang/lib/Basic/SourceManager.cpp b/clang/lib/Basic/SourceManager.cpp index 82096421f8579..1b4be1414b5d4 100644 --- a/clang/lib/Basic/SourceManager.cpp +++ b/clang/lib/Basic/SourceManager.cpp @@ -24,6 +24,7 @@ #include "llvm/ADT/StringRef.h" #include "llvm/ADT/StringSwitch.h" #include "llvm/Support/Allocator.h" +#include "llvm/Support/AutoConvert.h" #include "llvm/Support/Capacity.h" #include "llvm/Support/Compiler.h" #include "llvm/Support/Endian.h" @@ -586,17 +587,6 @@ SourceManager::getOrCreateFileID(FileEntryRef SourceFile, FileCharacter); } -/// Helper function to determine if an input file requires conversion -bool needConversion(StringRef Filename) { -#ifdef __MVS__ - llvm::ErrorOr<bool> NeedConversion = - llvm::needzOSConversion(Filename.str().c_str()); - return NeedConversion && *NeedConversion; -#else - return false; -#endif -} - /// createFileID - Create a new FileID for the specified ContentCache and /// include position. This works regardless of whether the ContentCache /// corresponds to a file or some other input source. @@ -616,8 +606,9 @@ FileID SourceManager::createFileIDImpl(ContentCache &File, StringRef Filename, return FileID::get(LoadedID); } unsigned FileSize = File.getSize(); - bool NeedConversion = needConversion(Filename); - if (NeedConversion) { + llvm::ErrorOr<bool> NeedConversion = + llvm::needConversion(Filename.str().c_str()); + if (NeedConversion && *NeedConversion) { // Buffer size may increase due to potential z/OS EBCDIC to UTF-8 // conversion. if (std::optional<llvm::MemoryBufferRef> Buffer = _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits