The technique used before relied on linear search of all builtins by
name. This did not work with qualified C++ names of C library names such
as std::pow and std::round.
Use ASTContext to get to BuiltinInfo, and in turn use that to check more
specific traits of the builtin by ID instead of name.
Fixes issue #918.
@carlosgalvezp @avostrik Could you take this for a spin and let me know if it solves your respective issues in context? Thanks!
@kimgr Just tested it and it works perfectly on my issue, thanks a lot! I wonder if it makes sense to add a unit test for this to make sure it doesn't pop up again?
@carlosgalvezp Thanks for testing. Oh, and thanks for keeping me honest -- I was so excited I got it to work that I forgot about regression testing :clown_face:. Will update shortly.