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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to