[PATCH] D65059: [ARM] Better OR's for MVE compares

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 22 00:58:28 PDT 2019


SjoerdMeijer added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMISelLowering.cpp:11850
+  unsigned Opposite1 = 0;
+  for (auto Code : InvertCodes) {
+    if (N0->getOpcode() == Code.Opcode)
----------------
I am wondering if a map is easier as a data structure here so that you can just do a lookup and don't need the for-loop here?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65059/new/

https://reviews.llvm.org/D65059





More information about the llvm-commits mailing list