[PATCH] D89252: [C API] Add getters and setters for NUW, NSW, and Exact

Benji Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 5 18:05:19 PDT 2023


Benjins added a comment.

Hi folks. I came across this review while looking at nsw/nuw/etc. information being missing from libraries that consume the C API bindings, e.g. Rust. I'd like to see this land, so is there anything holding this patch back? It sounds like it got thumbs-up from reviewers, so is there someone else who needs to merge it? I tried applying it to latest trunk just now and it still applies fine and the tests still pass. I assume it's still good to land?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89252



More information about the llvm-commits mailing list