[PATCH] D76231: Make ConstantOp::getAsmResultNames() handle signfull constants

River Riddle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 16 18:35:46 PDT 2020


rriddle requested changes to this revision.
rriddle added a comment.
This revision now requires changes to proceed.

Thanks for the patch! Unfortunately ATM, the standard dialect does not intend to support non-signless integers. https://mlir.llvm.org/docs/Rationale/#integer-signedness-semantics


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76231





More information about the llvm-commits mailing list