[PATCH] D53157: Teach the IRBuilder about constrained fadd and friends

Andy Kaylor via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 8 11:39:01 PST 2018


andrew.w.kaylor added a comment.

Craig Topper also raised some concerns with me (in person, his desk is right by mine) about the potential effect this might have on code size. He tells me that IRBuilder calls are intended to always be inlined and if we grow the implementation of these functions too much it could lead to noticeable bloat. It still seems to me like it might be worthwhile for the simplification it would allow in the front end, but I'm not really a front end guy so I definitely agree that we should get some input from front end people about what they want.


https://reviews.llvm.org/D53157





More information about the llvm-commits mailing list