[PATCH] D87279: [clang] Fix handling of physical registers in inline assembly operands.

Craig Topper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 6 23:35:09 PDT 2020


craig.topper added inline comments.


================
Comment at: clang/lib/CodeGen/CGStmt.cpp:2090
+        CGM.Error(S.getAsmLoc(), "Multiple outputs to hard register: " + GCCReg);
+      PhysRegOutputs.insert(GCCReg);
+    }
----------------
Can we get rid of the count call above and just use the bool from the std::pair returned by insert?


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

https://reviews.llvm.org/D87279



More information about the cfe-commits mailing list