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

Jonas Paulsson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 7 01:43:51 PDT 2020


jonpa added inline comments.


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



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

https://reviews.llvm.org/D87279



More information about the cfe-commits mailing list