Page MenuHomePhabricator

[SelectionDAG] Fix lowering of vector geps
ClosedPublic

Authored by jroelofs on Jul 29 2020, 12:18 PM.

Details

Summary

This fixes an assertion failure that was being triggered in SelectionDAG::getZeroExtendInReg(), where it was trying to extend the <2xi32> to i64 (which should have been <2xi64>).

Diff Detail

Unit TestsFailed

TimeTest
100 mswindows > Clang.OpenMP::declare_variant_device_isa_codegen_1.c
Script: -- : 'RUN: at line 1'; c:\ws\w32-1\llvm-project\premerge-checks\build\bin\clang.exe -cc1 -internal-isystem c:\ws\w32-1\llvm-project\premerge-checks\build\lib\clang\12.0.0\include -nostdsysteminc -verify -fopenmp -x c -triple x86_64-pc-windows-gnu -emit-llvm C:\ws\w32-1\llvm-project\premerge-checks\clang\test\OpenMP\declare_variant_device_isa_codegen_1.c -o - -fopenmp-version=50 | c:\ws\w32-1\llvm-project\premerge-checks\build\bin\filecheck.exe C:\ws\w32-1\llvm-project\premerge-checks\clang\test\OpenMP\declare_variant_device_isa_codegen_1.c --check-prefix=GENERIC

Event Timeline

jroelofs created this revision.Jul 29 2020, 12:18 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 29 2020, 12:18 PM
jroelofs requested review of this revision.Jul 29 2020, 12:18 PM
dmgreen accepted this revision.Jul 30 2020, 12:17 AM
dmgreen added a subscriber: dmgreen.

LGTM

This revision is now accepted and ready to land.Jul 30 2020, 12:17 AM
aemerson accepted this revision.Jul 30 2020, 10:56 AM
This revision was landed with ongoing or failed builds.Jul 30 2020, 1:57 PM
This revision was automatically updated to reflect the committed changes.