Fix some clang-tidy suggested issues 2

Author: alexander-penev

lib/Interpreter/CppInterOp.cpp

@@ -3584,10 +3584,10 @@ namespace Cpp {
 
     // Lazy materialization unit class helper
     class AutoloadLibraryMU : public llvm::orc::MaterializationUnit {
-      const std::string lib;
+      std::string lib;
       llvm::orc::SymbolNameVector syms;
     public:
-      AutoloadLibraryMU(const std::string Library, const llvm::orc::SymbolNameVector &Symbols)
+      AutoloadLibraryMU(const std::string& Library, const llvm::orc::SymbolNameVector &Symbols)
         : MaterializationUnit({getSymbolFlagsMap(Symbols), nullptr}), lib(Library), syms(Symbols) {}
 
       [[nodiscard]] StringRef getName() const override {

lib/Interpreter/DynamicLibraryManager.cpp

@@ -392,9 +392,9 @@ namespace Cpp {
       return;
 
     DyLibHandle dyLibHandle = nullptr;
-    for (DyLibs::const_iterator I = m_DyLibs.begin(), E = m_DyLibs.end(); I != E; ++I) {
-      if (I->second == canonicalLoadedLib) {
-        dyLibHandle = I->first;
+    for (const auto& dylib : m_DyLibs) {
+      if (dylib.second == canonicalLoadedLib) {
+        dyLibHandle = dylib.first;
         break;
       }
     }