[PATCH] D21809: [SystemZ] Add support for the .insn directive

Roger Ferrer Ibanez via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 10 08:12:52 PDT 2016


rogfer01 added a subscriber: rogfer01.

================
Comment at: llvm/trunk/lib/Target/SystemZ/AsmParser/SystemZAsmParser.cpp:512-520
@@ +511,11 @@
+
+// For equal_range comparison.
+struct CompareInsn {
+  bool operator() (const InsnMatchEntry &LHS, StringRef RHS) {
+    return LHS.Format < RHS;
+  }
+  bool operator() (StringRef LHS, const InsnMatchEntry &RHS) {
+    return LHS < RHS.Format;
+  }
+};
+
----------------
This fails to build in Windows with Visual Studio 2013 with debugging enabled because of some debugging macro disaster in the `std::equal_range` used below.

Do you mind if I add this extra `operator()` like below?
```
  bool operator() (const InsnMatchEntry &LHS, const InsnMatchEntry &RHS) {
	  return LHS.Format < RHS.Format;
  }
```

I know it looks silly but this fails to build from source otherwise when debugging is enabled.

Thank you


Repository:
  rL LLVM

https://reviews.llvm.org/D21809





More information about the llvm-commits mailing list