[PATCH] D85314: [IR] Add FixedPointBuilder.

John McCall via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 18 14:35:25 PDT 2020


rjmccall added a comment.

I'm assuming that you've extracted this code out of the corresponded Clang code and that there's nothing functionally wrong with it.  The interface looks fine.  Can we add LLVM tests for this, or that too difficult to fit into the existing test infrastructure, and we should just wait to test it by moving Clang to use it?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85314



More information about the llvm-commits mailing list