================ Comment at: lib/Frontend/CompilerInvocation.cpp:307 @@ -308,1 +306,3 @@ + << Args.getLastArg(OPT_O)->getAsString(Args) << OptLevel << DefaultOptLevel; + OptLevel = DefaultOptLevel; } ---------------- This seems slightly redundant. Either you use literal 3 all over, or set the DefaultOptLevel on a higher place. Also, this is not the "default" level, but the fall-back level for high levels, maybe a better word would be "MaxOptLevel", if there isn't one already (I don't know).
If there isn't, than maybe you could create a MaxOptLevel on a higher level and change all comparisons "OptLevel > 3" for "OptLevel > MaxOptLevel". http://llvm-reviews.chandlerc.com/D2125 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
