[PATCH] D67925: [FPEnv] Strict FP tests should use the requisite function attributes

Andy Kaylor via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 3 13:21:29 PDT 2019


andrew.w.kaylor accepted this revision.
andrew.w.kaylor added a comment.
This revision is now accepted and ready to land.

I assume you've tested this with your verifier changes. This is a bit too much to verify every call and function manually. I looked at the ones that weren't boilerplate updates, and I sampled the boilerplate changes. Based on that review strategy, it looks good to me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67925





More information about the llvm-commits mailing list