[PATCH] D14471: [AArch64] Fix a crash in driver
Renato Golin via cfe-commits
cfe-commits at lists.llvm.org
Thu Nov 12 02:54:25 PST 2015
rengolin added a comment.
In http://reviews.llvm.org/D14471#287635, @ahatanak wrote:
> I didn't include a test case because I didn't know how to write a test that passes on an aarch64 host and fails on anything else. Do you know of any test cases in trunk that pass or fail depending on which host it is running on?
It's possible to do something with the python config, though I don't know how.
> Wouldn't it break on an aarch64 host? With "-mtune=native", the current code in trunk will get the host cpu name (which I believe is currently always "generic" for aarch64). But if I apply this patch, it will error out because "native" is not a valid cpu name.
That's my point. It shouldn't break.
If you can get the CPU name, return it. If not, return "native". If you're on AArch64 and you can't get the CPU name, that's a bug that needs fixing. If it's always "generic", that's another piece of code that needs fixing. If whatever getCPU function you use doesn't return a valid name, and the name chosen was "native", you should return "native".
--renato
http://reviews.llvm.org/D14471
More information about the cfe-commits
mailing list