[PATCH] Prevent clang from throwing the diagnostics twice.

Umesh Kalappa umesh.kalappa0 at gmail.com
Fri Nov 15 03:35:40 PST 2013


Hi All,

Clang pop up with the  "error: invalid integral value" diagnostics  twice
,when you enable the optimization  through  -O with a  non-integer value
 i.e -O<non integer>  as show below

$ clang -Of -S test.c
error: invalid integral value 'f' in '-Of'
error: invalid integral value 'f' in '-Of'


Attached patch fix the issue as

$ clang -Of -S test.c
error: invalid integral value 'f' in '-Of'


Clang version
clang version 3.4
(http://llvm.org/git/clang.git26297f57634994b4ae47a0774c372d6944265bb2)
(
http://llvm.org/git/llvm.git 868e74bdce7a2c49d60e2ef56a077b0aa7f2ba08)

Target: i386-pc-linux-gnu
Thread model: posix

Builded successfully on trunk and is it ok to commit ??


Thanks
~Umesh
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131115/728e6fce/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: clang_changes.patch
Type: application/octet-stream
Size: 1003 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131115/728e6fce/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: llvm_changes.patch
Type: application/octet-stream
Size: 2066 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131115/728e6fce/attachment-0001.obj>


More information about the cfe-commits mailing list