Details
Details
- Reviewers
rnk aaron.ballman bogner
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
Adding -Wno-cast-function-type seems reasonable, but we should do it more carefully.
cmake/modules/HandleLLVMOptions.cmake | ||
---|---|---|
583–586 | You should put this here and follow this pattern instead to avoid setting the -Wno- flag on old versions of GCC that don't support it. GCC normally does not warn about unrecognized -Wno- flags unless it emits a warning, in which case it warns about all the unrecognized -Wno- flags. That's less than ideal. |
You should put this here and follow this pattern instead to avoid setting the -Wno- flag on old versions of GCC that don't support it. GCC normally does not warn about unrecognized -Wno- flags unless it emits a warning, in which case it warns about all the unrecognized -Wno- flags. That's less than ideal.