[llvm-dev] Sincos for X86_64's GNUX32 and ARM's GNUEABI/GNUEABIHF enviroments

Daniel Sanders via llvm-dev llvm-dev at lists.llvm.org
Wed Jun 15 03:24:41 PDT 2016


While writing http://reviews.llvm.org/D20916, I stumbled across some code affecting ARM and X86_64 environments that looks like it might be unintentional. I thought I should ask about it here since that patch has a '[mips]' tag and therefore might not be noticed by someone who knows.

I've noticed that the GNUX32 and GNUEABI/GNUEABIHF environments don't make use of the sincos libcall like the GNU environment does because the guarding condition for this is 'TT.getEnvironment() == Triple::GNU'. The comment in canCombineSinCosLibcall() in LegalizeDAG.cpp suggests that it's intending to use sincos for all GNU environments so I'm wondering whether the exclusion of GNUX32/GNUEABI/GNUEABIHF is deliberate or not.

Can anyone confirm?


More information about the llvm-dev mailing list