[PATCH] D51145: Guard FMF context by excluding some FP operators from FPMathOperator

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 11 15:15:24 PDT 2018


spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

LGTM - I do wonder if it would be better to white-list just the FP ops that we know can carry FMF though (ie, make this consistent with IR verification)...unless FPMathOperator is used for some other purposes.


https://reviews.llvm.org/D51145





More information about the llvm-commits mailing list