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 39151 - [X86][SSE] Merge rL343373 into 7.0.1 - Fixed issue with v2i64 variable shifts on 32-bit targets
Summary: [X86][SSE] Merge rL343373 into 7.0.1 - Fixed issue with v2i64 variable shifts...
Status: RESOLVED FIXED
Alias: None
Product: libraries
Classification: Unclassified
Component: Backend: X86 (show other bugs)
Version: trunk
Hardware: PC Windows NT
: P enhancement
Assignee: Craig Topper
URL:
Keywords:
Depends on:
Blocks: release-7.0.1
  Show dependency tree
 
Reported: 2018-10-02 03:48 PDT by Simon Pilgrim
Modified: 2018-10-19 12:57 PDT (History)
4 users (show)

See Also:
Fixed By Commit(s): r343373 r344810

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 Simon Pilgrim 2018-10-02 03:48:36 PDT 
[X86][SSE] Fixed issue with v2i64 variable shifts on 32-bit targets

The shift amount might have peeked through a extract_subvector, altering the number of vector elements in the 'Amt' variable - so we were incorrectly calculating the ratio when peeking through bitcasts, resulting in incorrectly detecting splats.
Comment 1 Tom Stellard 2018-10-19 12:18:22 PDT 
Hi Craig,

Is this OK to merge?

https://reviews.llvm.org/rL343373
Comment 2 Craig Topper 2018-10-19 12:20:57 PDT 
Yes, this is ok to merge.
Comment 3 Tom Stellard 2018-10-19 12:57:05 PDT 
Merged: r344810