[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