================
Comment at: lib/Support/CommandLine.cpp:1511
@@ -1497,5 +1510,3 @@
   static bool OptionCategoryCompare(OptionCategory *A, OptionCategory *B) {
-    int Length = strcmp(A->getName(), B->getName());
-    assert(Length != 0 && "Duplicate option categories");
-    return Length < 0;
+    return strcmp(A->getName(), B->getName());
   }
----------------
Alex McCarthy wrote:
> This used to return true if strcmp returned a negative value, but I think it 
> now returns true for any non-zero return value. Is this correct, or should 
> this be return strcmp() < 0?
Good catch! Thanks!


http://llvm-reviews.chandlerc.com/D2699
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to