xry111 added a comment. In D155824#4521047 <https://reviews.llvm.org/D155824#4521047>, @SixWeining wrote:
> In D155824#4518597 <https://reviews.llvm.org/D155824#4518597>, @xry111 wrote: > >> Do we need to convert Xuerui's label alignment change to use the -mtune >> framework? > > How to set the numbers if `TuneCPU` is empty? If there is no differences > among `empty`, `loongarch64` and `la464`, seems it's better to do this until > we support other uarchs, e.g. `la[236]64`. > > Just like the `TODO` in D148622 <https://reviews.llvm.org/D148622>: > > 63 // TODO: Check TuneCPU and override defaults (that are for LA464) once > we > 64 // support optimizing for more uarchs. In GCC I added the numbers for "loongarch64" and "la464" although they are the same, and GCC does not have "empty". But yes we can do things later here. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155824/new/ https://reviews.llvm.org/D155824 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
