[PATCH] D61555: Remove BinaryOpsEnd case from ConstantFoldBinaryInstruction(...)

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 5 08:16:30 PDT 2019


spatel added a comment.

In D61555#1490960 <https://reviews.llvm.org/D61555#1490960>, @cameron.mcinally wrote:

> Oh, a proper warning at compile time. That makes sense. I'm okay with abandoning this patch if Sanjay is okay with it.


That's fine; I wasn't thinking about compile-time warning vs. run-time assert.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61555





More information about the llvm-commits mailing list