[PATCH] D22678: [mips][ias] Enable IAS by default for N64 on Debian mips64el.

Daniel Sanders via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 22 07:45:05 PDT 2016


dsanders added a comment.

In https://reviews.llvm.org/D22678#492835, @mpf wrote:

> I'm not sure there is any subtlety between all n64 and debian mips64/mips64el. If someone uses a manually modified triple to have gnuabi64 along with a -mabi=64 option then n64 will get IAS enabled. If you use a 32bit triple with a -mabi=64 then it won't be enabled.
>
> I would change the comment to be:
>
> // Enable IAS by default if the environment encodes the ABI explicitly
>
> Matthew


I'm currently treating support for Debian mips64el and the more general ABI-in-triples work as fully independent pieces of work. The current comment is intended to be correct for the current LLVM trunk and I was planning to change the comment to something like your suggestion when the ABI-in-triples work lands and I enable IAS for N64 in general.


https://reviews.llvm.org/D22678





More information about the llvm-commits mailing list