[clang] [llvm] [clang][llvm][SPIR-V] Explicitly encode native integer widths for SPIR-V (PR #110695)
Dmitry Sidorov via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 11 04:15:00 PDT 2024
MrSidims wrote:
> InstCombine's primary function is a canonicalization pass. You shouldn't be modifying it for specifically SPIRV optimizations (with the exception of SPIRV intrinsic support). SPIRV specific transforms belong in later backend IR passes
Does it mean, that the reasoning behind this very PR is not legit?
> This is problematic as it leads to optimisation passes, such as InstCombine, getting ideas and e.g. shrinking to non byte-multiple integer types, which is not desirable and can lead to breakage further down in the toolchain.
https://github.com/llvm/llvm-project/pull/110695
More information about the cfe-commits
mailing list