[PATCH] D92140: Fix noderef for array member of deref expr

2020-12-07 Thread Jann Horn via Phabricator via cfe-commits
thejh added a comment. Thanks! :) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92140/new/ https://reviews.llvm.org/D92140 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/c

[PATCH] D92140: Fix noderef for array member of deref expr

2020-12-07 Thread Leonard Chan via Phabricator via cfe-commits
leonardchan closed this revision. leonardchan added a comment. Committed in 155fca3cae275562e626d3e4fbfac70b4b75d2e7 . (Sorry, I forgot to change the author email in the commit) Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D92140: Fix noderef for array member of deref expr

2020-12-07 Thread Leonard Chan via Phabricator via cfe-commits
leonardchan added a comment. Sorry. I accidentally missed this in my emails. Will commit these. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92140/new/ https://reviews.llvm.org/D92140 ___ cfe-commits ma

[PATCH] D92140: Fix noderef for array member of deref expr

2020-12-01 Thread Jann Horn via Phabricator via cfe-commits
thejh added a comment. @leonardchan I don't have commit access; can you land this change and D92141 for me? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92140/new/ https://reviews.llvm.org/D92140 __

[PATCH] D92140: Fix noderef for array member of deref expr

2020-11-25 Thread Leonard Chan via Phabricator via cfe-commits
leonardchan accepted this revision. leonardchan added a comment. This revision is now accepted and ready to land. Thanks for catching this. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92140/new/ https://reviews.llvm.org/D92140 _

[PATCH] D92140: Fix noderef for array member of deref expr

2020-11-25 Thread Jann Horn via Phabricator via cfe-commits
thejh created this revision. thejh added reviewers: leonardchan, aaron.ballman. Herald added a project: clang. Herald added a subscriber: cfe-commits. thejh requested review of this revision. Given an __attribute__((noderef)) pointer "p" to the struct struct s { int a[2]; }; ensure that the fo