[PATCH] D28543: Elliminates uninitialized warning for volitile varibles.

Joerg Sonnenberger via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 10 14:41:13 PST 2017


joerg added a comment.

Please fix the spelling errors in the titel / summary before commit.

I somewhat agree with Hal -- I think this is too aggressive. Common use cases for local volatile include atomic ops or returns-twice functions like setjmp/longjmp.
Disabling the warning in those cases has a high chance of hiding real problems. I would find it much more useful to move this case into a warning subgroup, so that
it can be selectively disabled by command line or pragma.


https://reviews.llvm.org/D28543





More information about the cfe-commits mailing list