LLVM Bugzilla is read-only and represents the historical archive of all LLVM issues filled before November 26, 2021. Use github to submit LLVM bugs

Bug 32409 - Merge r298604 into the 4.0 branch : [SDAG] Fix zeroExtend assertion error
Summary: Merge r298604 into the 4.0 branch : [SDAG] Fix zeroExtend assertion error
Status: RESOLVED FIXED
Alias: None
Product: new-bugs
Classification: Unclassified
Component: new bugs (show other bugs)
Version: 4.0
Hardware: All All
: P enhancement
Assignee: Tom Stellard
URL: https://reviews.llvm.org/rL298604
Keywords:
Depends on:
Blocks: release-4.0.1
  Show dependency tree
 
Reported: 2017-03-24 08:11 PDT by Tom Stellard
Modified: 2017-04-24 13:14 PDT (History)
2 users (show)

See Also:
Fixed By Commit(s):

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Tom Stellard 2017-03-24 08:11:00 PDT 
Is this patch OK to merge to the 4.0 branch?
Comment 1 Nirav Dave 2017-04-04 11:39:27 PDT 
PR32284 was reopened because of similar cases. If we add this we should also consider: https://reviews.llvm.org/D31625
Comment 2 Justin Bogner 2017-04-04 13:09:21 PDT 
Change looks good, though it's a bit unfortunate the test is just a "don't crash" test that doesn't actually check anything.
Comment 3 Tom Stellard 2017-04-21 07:27:27 PDT 
Merged: r300979

Nirav once D31625 is committed, can you create a separate merge request for that one?
Comment 4 Nirav Dave 2017-04-24 13:14:31 PDT 
Will do.