[llvm] [SandboxIR] Implement CallBase and CallInst (PR #100218)

Alina Sbirlea via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 26 10:14:41 PDT 2024


================
@@ -435,7 +453,7 @@ class Constant : public sandboxir::User {
     return getUseOperandNoDefault(Use);
   }
 #ifndef NDEBUG
-  void verify() const final {
+  void verify() const override {
     assert(isa<llvm::Constant>(Val) && "Expected Constant!");
   }
   friend raw_ostream &operator<<(raw_ostream &OS,
----------------
alinas wrote:

Yes, that's what I meant, currently the operator calls dump so it can't be taken outside of the NDEBUG as is. I didn't see the operator<< for the Load/Store instructions, but it was in the full patch (in the branch). Ok to revisit in separate patch.

https://github.com/llvm/llvm-project/pull/100218


More information about the llvm-commits mailing list