<div dir="ltr"><div>Sorry about that! I am reverting the change for now. Waiting for the tests to pass locally with the change reverted.<br></div><div><br></div><div>Sameer.<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 21, 2018 at 9:20 PM, Michel Dänzer via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">michel.daenzer added a comment.<br>
<br>
This broke glamor with the radeonsi driver, hits unreachable():<br>
<br>
Pass ID not registered<br>
UNREACHABLE executed at ../lib/CodeGen/<wbr>TargetPassConfig.cpp:536!<br>
<br>
Thread 1 "X" received signal SIGABRT, Aborted.<br>
__GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/<wbr>raise.c:51<br>
51      ../sysdeps/unix/sysv/linux/<wbr>raise.c: No such file or directory.<br>
(gdb) bt<br>
#0  __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/<wbr>raise.c:51<br>
#1  0x00007ffff67a12f1 in __GI_abort () at abort.c:79<br>
#2  0x00007ffff0c9d3c3 in llvm::llvm_unreachable_<wbr>internal (msg=<optimized out>, file=<optimized out>, line=<optimized out>) at ../lib/Support/ErrorHandling.<wbr>cpp:222<br>
#3  0x00007ffff11cbfce in llvm::TargetPassConfig::<wbr>addPass (this=0x555555856560, PassID=<optimized out>, verifyAfter=true, printAfter=true) at ../lib/CodeGen/<wbr>TargetPassConfig.cpp:536<br>
#4  0x00007ffff2075582 in (anonymous namespace)::AMDGPUPassConfig::<wbr>addPreISel (this=0x555555856560) at ../lib/Target/AMDGPU/<wbr>AMDGPUTargetMachine.cpp:682<br>
#5  (anonymous namespace)::GCNPassConfig::<wbr>addPreISel (this=0x555555856560) at ../lib/Target/AMDGPU/<wbr>AMDGPUTargetMachine.cpp:750<br>
#6  0x00007ffff11cc4bb in llvm::TargetPassConfig::<wbr>addISelPrepare (this=0x555555856560) at ../lib/CodeGen/<wbr>TargetPassConfig.cpp:693<br>
#7  0x00007ffff11cc8bf in llvm::TargetPassConfig::<wbr>addISelPasses (this=0x555555856560) at ../lib/CodeGen/<wbr>TargetPassConfig.cpp:773<br>
#8  0x00007ffff0fcaa65 in addPassesToGenerateCode (TM=0x55555584a320, PM=..., DisableVerify=<optimized out>, WillCompleteCodeGenPipeline=@<wbr>0x7fffffffda47: true, Out=..., MMI=0x5555558608c0) at ../lib/CodeGen/<wbr>LLVMTargetMachine.cpp:113<br>
#9  0x00007ffff0fca968 in llvm::LLVMTargetMachine::<wbr>addPassesToEmitFile (this=<optimized out>, PM=..., Out=..., DwoOut=<optimized out>, FileType=llvm::TargetMachine::<wbr>CGFT_ObjectFile, DisableVerify=false, MMI=0x0) at ../lib/CodeGen/<wbr>LLVMTargetMachine.cpp:205<br>
#10 0x00007ffff4501dbc in ac_create_llvm_passes () at ../../../src/amd/common/ac_<wbr>llvm_helper.cpp:134<br>
#11 0x00007ffff43f9669 in si_init_compiler (sscreen=sscreen@entry=<wbr>0x5555558400a0, compiler=compiler@entry=<wbr>0x5555558407b0) at ../../../../../src/gallium/<wbr>drivers/radeonsi/si_pipe.c:128<br>
#12 0x00007ffff43fb792 in radeonsi_screen_create (ws=<optimized out>, config=<optimized out>) at ../../../../../src/gallium/<wbr>drivers/radeonsi/si_pipe.c:<wbr>1079<br>
#13 0x00007ffff44cdbbb in amdgpu_winsys_create (fd=fd@entry=16, config=config@entry=<wbr>0x7fffffffdc18, screen_create=screen_create@<wbr>entry=0x7ffff43fb200 <radeonsi_screen_create>) at ../../../../../../src/gallium/<wbr>winsys/amdgpu/drm/amdgpu_<wbr>winsys.c:351<br>
#14 0x00007ffff3cb0cb2 in pipe_radeonsi_create_screen (fd=16, config=0x7fffffffdc18) at ../../../../../src/gallium/<wbr>auxiliary/target-helpers/drm_<wbr>helper.h:171<br>
#15 0x00007ffff42f764d in pipe_loader_create_screen (dev=0x55555583be00) at ../../../../../src/gallium/<wbr>auxiliary/pipe-loader/pipe_<wbr>loader.c:137<br>
#16 0x00007ffff4133bfe in dri2_init_screen (sPriv=0x55555583a1c0) at ../../../../../src/gallium/<wbr>state_trackers/dri/dri2.c:2112<br>
#17 0x00007ffff412e011 in driCreateNewScreen2 (scrn=0, fd=13, extensions=<optimized out>, driver_extensions=<optimized out>, driver_configs=0x5555557f8c58, data=0x5555557f8a90) at ../../../../../../src/mesa/<wbr>drivers/dri/common/dri_util.c:<wbr>153<br>
#18 0x00007ffff57a556e in dri_screen_create_dri2 (dri=dri@entry=0x5555557f8a90, driver_name=<optimized out>) at ../../../src/gbm/backends/dri/<wbr>gbm_dri.c:451<br>
#19 0x00007ffff57a5922 in dri_screen_create (dri=0x5555557f8a90) at ../../../src/gbm/backends/dri/<wbr>gbm_dri.c:526<br>
#20 dri_device_create (fd=13) at ../../../src/gbm/backends/dri/<wbr>gbm_dri.c:1433<br>
#21 0x00007ffff57a31a9 in gbm_create_device (fd=13) at ../../../src/gbm/main/gbm.c:<wbr>137<br>
#22 0x00007ffff57c0b22 in AMDGPUPreInitAccel_KMS (pScrn=0x5555557f62a0) at ../../src/amdgpu_kms.c:1167<br>
#23 AMDGPUPreInit_KMS (pScrn=0x5555557f62a0, flags=<optimized out>) at ../../src/amdgpu_kms.c:1394<br>
#24 0x00005555555ef898 in InitOutput (pScreenInfo=pScreenInfo@entry<wbr>=0x5555557b5fa0 <screenInfo>, argc=argc@entry=4, argv=argv@entry=<wbr>0x7fffffffeb18) at ../../../../hw/xfree86/common/<wbr>xf86Init.c:483<br>
#25 0x00005555555b0a88 in dix_main (argc=4, argv=0x7fffffffeb18, envp=<optimized out>) at ../../dix/main.c:191<br>
#26 0x00007ffff678cb17 in __libc_start_main (main=0x55555559a1c0 <main>, argc=4, argv=0x7fffffffeb18, init=<optimized out>, fini=<optimized out>, rtld_fini=<optimized out>, stack_end=0x7fffffffeb08) at ../csu/libc-start.c:310<br>
#27 0x000055555559a1fa in _start ()<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D52221" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D52221</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div>