[PATCH] D82317: [Clang/Test]: Update tests where `noundef` attribute is necessary
Johannes Doerfert via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 11 19:42:29 PDT 2020
jdoerfert added a comment.
In D82317#2211643 <https://reviews.llvm.org/D82317#2211643>, @guiand wrote:
> After discussing with @eugenis, for the meantime it might be best to do the following:
>
> - Change the masking attribute to be `-fdisable-noundef-analysis` (name notwithstanding), and have it completely turn off all `noundef`s
> - Change the llvm-lit configuration to use the new codegen flag for all the tests by default
> - Have `noundef` emitted in the frontend by default (when the codegen flag isn't present)
TBH, I don't see how this solves any problem. It just makes it a problem for someone in the future...
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82317/new/
https://reviews.llvm.org/D82317
More information about the cfe-commits
mailing list