[PATCH] D19419: [ELF] - Implemented comparsion operators for linkerscript

[George Rimar via llvm-commits]

grimar added inline comments.

================
Comment at: ELF/LinkerScript.cpp:152
@@ -135,1 +151,3 @@
+  if (Op == "!=")
+    return L == R ? 0 : 1;
   llvm_unreachable("invalid operator");
----------------
emaste wrote:
> Trivial comment, but why not `L != R` here? And, why not just `return L < R;`, `return L > R;` etc. in general? For me at least `? 1 : 0` is not any more clear/readable.
> 
I think you right. I`ll change that.


http://reviews.llvm.org/D19419