[llvm] [GlobalIsel] Visit ICmp (PR #105991)
Thorsten Schütt via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 3 10:27:48 PDT 2024
tschuett wrote:
> Some more comments. Frankly I'm having trouble following some of your comments like this:
>
> > IR is in perfect canonical state. Codegen may do some target specific optimizations, but I would be surprised if they move constants to the left. The IRTranslator faithfully translates IR into gMIR. Thus porting middle-end optimizations to the backend is a waste of time because they cannot give any size improvements.
>
> Are you agreeing or disagreeing with your current patch? At this point I have no idea.
I am supportive of this patch. I was only stating the common assumption. But also the cmp x, 0 is at odds with this statement.
https://github.com/llvm/llvm-project/pull/105991
More information about the llvm-commits
mailing list