[PATCH] D105169: [Clang/Test]: Rename enable_noundef_analysis to disable-noundef-analysis and turn it off by default

Nuno Lopes via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 20 04:40:23 PST 2022

nlopes added a comment.

In D105169#3257849 <https://reviews.llvm.org/D105169#3257849>, @sdesmalen wrote:

> In D105169#3254757 <https://reviews.llvm.org/D105169#3254757>, @nlopes wrote:
>> In D105169#3254692 <https://reviews.llvm.org/D105169#3254692>, @DavidSpickett wrote:
>>> Hi, this patch has caused a gcc test suite failure on our SVE bots: https://lab.llvm.org/buildbot/#/builders/184/builds/1941
>>> (test source is https://github.com/llvm/llvm-test-suite/blob/main/SingleSource/Regression/C/gcc-c-torture/execute/pr79286.c)
>>> I've only just been made aware of the failure so we will spend some time to find out the reason before asking that anything be changed. Could be something specific to SVE in which case we'll handle it.
>> Thanks!
>> That test has UB in the printf class as it reads a non-initialized variable. Can you check if changing line 6 to 'int e = 0;' solves the problem?
> I can confirm that fixes the issue.

Thank you! I've committed the fix.

  rG LLVM Github Monorepo



More information about the cfe-commits mailing list