[llvm] [llvm][RISCV] Tune lowerSelect on zicond for code size (PR #210026)

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 16 08:40:41 PDT 2026


topperc wrote:

> Sorry, removing my LGTM for the moment.
> 
> This seems like somewhat of a duplicate of #163501 (sorry, confused myself with which link i posted) - but with a slightly different condition.
> 
> Can we check this is covering both Os and Oz? I think right now it only covers Oz.

Do we want to disable Zicond for Os? Zicond can prevent a mispredicted branch. How much do we care about that for Os?

https://github.com/llvm/llvm-project/pull/210026


More information about the llvm-commits mailing list