[PATCH] Don't set ObjCAutoRefCount in original CI

Alp Toker alp at nuanti.com
Sun Jul 6 15:35:27 PDT 2014


Hi Fariborz,

As far as I can tell the original CI is immutable at this point, so 
changing it isn't needed or desirable at this point. OK to remove this 
line? No change in test results.

Alp.

-- 
http://www.nuanti.com
the browser experts

-------------- next part --------------
diff --git a/lib/ARCMigrate/ARCMT.cpp b/lib/ARCMigrate/ARCMT.cpp
index 8a13b2e..9d525ff 100644
--- a/lib/ARCMigrate/ARCMT.cpp
+++ b/lib/ARCMigrate/ARCMT.cpp
@@ -369,7 +369,6 @@ static bool applyTransforms(CompilerInvocation &origCI,
                             DiagClient, /*ShouldOwnClient=*/false));
 
   if (outputDir.empty()) {
-    origCI.getLangOpts()->ObjCAutoRefCount = true;
     return migration.getRemapper().overwriteOriginal(*Diags);
   } else {
     return migration.getRemapper().flushToDisk(outputDir, *Diags);


More information about the cfe-commits mailing list