aprantl marked an inline comment as done.
aprantl added inline comments.

================
Comment at: clang/lib/CodeGen/CodeGenFunction.cpp:384
+      else
+        assert(EndLoc.isValid() && "no location for inlineable cleanup calls");
+    }
----------------
vsk wrote:
> The braces around the `else` block are probably needed in the NDEBUG build.
No, this should be fine. The `;` is not part of the macro.


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

https://reviews.llvm.org/D71042



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to