We were needlessly putting known constant values on the LHS of a G_MUL, which is suboptimal.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
| Paths 
 |  Differential  D73650  
[GlobalISel][IRTranslator] Follow convention and put constant offset of getelementptr arithmetic on RHS. ClosedPublic Authored by aemerson on Jan 29 2020, 11:20 AM. 
Details Summary We were needlessly putting known constant values on the LHS of a G_MUL, which is suboptimal. 
Diff Detail 
 Event TimelineHerald added subscribers: kerbowa, Petar.Avramovic, volkan and 6 others.  ·  View Herald TranscriptJan 29 2020, 11:21 AM This revision is now accepted and ready to land.Jan 29 2020, 11:22 AM Closed by commit rG0da937bb5c2b: [GlobalISel][IRTranslator] Follow convention and put constant offset of… (authored by aemerson).  ·  Explain WhyJan 29 2020, 11:40 AM This revision was automatically updated to reflect the committed changes. 
Revision Contents 
 
Diff 241243 llvm/lib/CodeGen/GlobalISel/IRTranslator.cpp
 llvm/test/CodeGen/AArch64/GlobalISel/arm64-irtranslator-gep.ll
 llvm/test/CodeGen/AArch64/GlobalISel/arm64-irtranslator-switch.ll
 llvm/test/CodeGen/AArch64/GlobalISel/translate-gep.ll
 llvm/test/CodeGen/AMDGPU/GlobalISel/irtranslator-getelementptr.ll
 llvm/test/CodeGen/AMDGPU/GlobalISel/lds-relocs.ll
 llvm/test/CodeGen/AMDGPU/GlobalISel/llvm.amdgcn.atomic.dec.ll
 llvm/test/CodeGen/AMDGPU/GlobalISel/llvm.amdgcn.atomic.inc.ll
 llvm/test/CodeGen/AMDGPU/GlobalISel/llvm.amdgcn.is.private.ll
 llvm/test/CodeGen/AMDGPU/GlobalISel/llvm.amdgcn.is.shared.ll
 llvm/test/CodeGen/AMDGPU/GlobalISel/llvm.amdgcn.update.dpp.ll
 llvm/test/CodeGen/AMDGPU/GlobalISel/mubuf-global.ll
 llvm/test/CodeGen/X86/GlobalISel/add-ext.ll
 llvm/test/CodeGen/X86/GlobalISel/ptr-add.ll
 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||