[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