[PATCH] D39304: [IR] redefine 'reassoc' fast-math-flag and add 'trans' fast-math-flag

[Sanjay Patel via Phabricator via llvm-commits]

spatel added reviewers: anemet, yuyichao.
spatel added a comment.

https://reviews.llvm.org/D39319 (Set contract flag when setting unsafe algebra flag) proposes to fix a bug introduced with https://reviews.llvm.org/D31164 (Add AllowContract to FastMathFlags). That bug is also fixed here - but I should add that test.

This might suggest that we should split 'trans' into its own small follow-up patch. That would also allow the unsafe TODO fixes to proceed independently of 'trans' in case that takes longer to sort out. Let me know if splitting this patch up seems better.


https://reviews.llvm.org/D39304