This is an archive of the discontinued LLVM Phabricator instance.

[clang][Interp] Handle DiscardResult for DeclRef- and ParenExprs
ClosedPublic

Authored by tbaeder on Apr 7 2023, 10:29 PM.

Details

Summary
Fixes https://github.com/llvm/llvm-project/issues/62004

Diff Detail

Event Timeline

tbaeder created this revision.Apr 7 2023, 10:29 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 7 2023, 10:29 PM
tbaeder requested review of this revision.Apr 7 2023, 10:29 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 7 2023, 10:29 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
tbaeder added inline comments.Apr 7 2023, 10:30 PM
clang/lib/AST/Interp/ByteCodeExprGen.cpp
197

Next time this pattern shows up, I need to add a convenience function.

tbaeder updated this revision to Diff 515660.Apr 21 2023, 2:33 AM
shafik accepted this revision.May 2 2023, 8:19 AM

LGTM

This revision is now accepted and ready to land.May 2 2023, 8:19 AM