We need to use oclc_daz_opt_on.amdgcn.bc bitcode when option fcuda-flush-denormal-to-zero is enabled for HIP.
Details
Details
Diff Detail
Diff Detail
- Repository
- rC Clang
Event Timeline
Comment Actions
I think we should have a lit test for linked bitcode files.
lib/Driver/ToolChains/HIP.cpp | ||
---|---|---|
78 | you can use StringRef here. Please use a more meaningful name, e.g. FlushDenormalControlBC. |
Comment Actions
I've changed the string to a more meaningful name, and also added a lit test to check that oclc_daz_opt_on.amdgcn.bc is used.
This comment was removed by ashi1.
lib/Driver/ToolChains/HIP.cpp | ||
---|---|---|
85 | we need to fix the order. should be ocml, then ockl, then irif. but should do it in another patch |
Comment Actions
I've updated. Moved the bitcode files into hip_dev_lib and added test for both on and off flush denorms to zero option.
Comment Actions
better merge these two tests and rename it as hip-device-libs.hip. We can also add other device lib tests to it later.
you can use StringRef here. Please use a more meaningful name, e.g. FlushDenormalControlBC.