Skip to content

builld: Enable -fanalyzer to catch more program error#8590

Closed
xiaoxiang781216 wants to merge 1 commit intoapache:masterfrom
xiaoxiang781216:compiler
Closed

builld: Enable -fanalyzer to catch more program error#8590
xiaoxiang781216 wants to merge 1 commit intoapache:masterfrom
xiaoxiang781216:compiler

Conversation

@xiaoxiang781216
Copy link
Copy Markdown
Contributor

@xiaoxiang781216 xiaoxiang781216 commented Feb 20, 2023

Summary

See discussion: apache/nuttx-apps#1576

Impact

catch more error

Testing

CI

@xiaoxiang781216 xiaoxiang781216 marked this pull request as draft February 20, 2023 08:10
@xiaoxiang781216 xiaoxiang781216 force-pushed the compiler branch 2 times, most recently from 2b0cbff to 851aa91 Compare February 20, 2023 12:29
@fjpanag
Copy link
Copy Markdown
Contributor

fjpanag commented Feb 21, 2023

@xiaoxiang781216 -fanalyzer is notorious for its false positives.
Can we make it a Kconfig option? So it can be disabled when it becomes annoying...

@xiaoxiang781216 xiaoxiang781216 force-pushed the compiler branch 2 times, most recently from 0663600 to 3640164 Compare February 23, 2023 15:59
@xiaoxiang781216
Copy link
Copy Markdown
Contributor Author

@xiaoxiang781216 -fanalyzer is notorious for its false positives. Can we make it a Kconfig option? So it can be disabled when it becomes annoying...

let me fix the warning first:).

apache/nuttx-apps#1576

Signed-off-by: Xiang Xiao <xiaoxiang@xiaomi.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants