nikic wrote:
@dtcxzyw I'm going to merge this after LLVM 18 branches next week.
https://github.com/llvm/llvm-project/pull/68882
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
dtcxzyw wrote:
Ping?
https://github.com/llvm/llvm-project/pull/68882
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2282,6 +2282,15 @@ Instruction
*InstCombinerImpl::visitGetElementPtrInst(GetElementPtrInst ) {
if (MadeChange)
return
+ // Canonicalize constant GEPs to i8 type.
brunodf-snps wrote:
Reading about the [planned
nikic wrote:
> > arrayidx
>
> We should teach `foldCmpLoadFromIndexedGlobal` to handle constant GEPs with
> i8 source element type.
There is a pending patch related to this:
https://github.com/llvm/llvm-project/pull/67093
https://github.com/llvm/llvm-project/pull/68882
dtcxzyw wrote:
> arrayidx
We should teach `foldCmpLoadFromIndexedGlobal` to handle constant GEPs with i8
source element type.
https://github.com/llvm/llvm-project/pull/68882
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
dtcxzyw wrote:
> @nikic Could you please have a look at
> [dtcxzyw/llvm-opt-benchmark#17](https://github.com/dtcxzyw/llvm-opt-benchmark/pull/17)?
> One regression:
>
> ```
> diff --git a/bench/brotli/optimized/compound_dictionary.c.ll
> b/bench/brotli/optimized/compound_dictionary.c.ll
>
https://github.com/aeubanks approved this pull request.
I think this is a good direction to go, lgtm
https://github.com/llvm/llvm-project/pull/68882
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
