[PATCH] D117293: enable noundef analysis with -fsanitize-memory-param-retval

Vitaly Buka via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 14 13:10:40 PST 2022


vitalybuka added a comment.

In D117293#3244668 <https://reviews.llvm.org/D117293#3244668>, @kda wrote:

> In D117293#3243735 <https://reviews.llvm.org/D117293#3243735>, @vitalybuka wrote:
>
>> Nice!
>>
>>>> This completes the work found in:
>>
>> It's too late for this set of patches, but phabricator supports "Depends on"  comments (I guess only of the first patch upload), and manual editing of "parent/child" revisions. Check "Edit Related Revisions..." in the top right corner.
>> So you can create convenient "Stack" tab like in liihttps://reviews.llvm.org/D117286
>
> I added "Depends On: D116633 <https://reviews.llvm.org/D116633>"  (but I am confused because it appears crossed out.)

Crossed out means submitted. It's OK.
It should be "Depends On D116633 <https://reviews.llvm.org/D116633>", no ":" (at least that's how I do and it works)
I guess it will work only on the first patch upload, After then you need to  "Edit Related Revisions..." manually.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117293/new/

https://reviews.llvm.org/D117293



More information about the cfe-commits mailing list