[PATCH] D45240: [ARM] Compute a target feature which corresponds to the ARM version.

Renato Golin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 6 05:04:28 PDT 2018


rengolin added a subscriber: joerg.
rengolin added a comment.

LTO is something I never considered before in the context of the target parser, but I understand the issues are similar to what the build attributes were trying to solve, so adding more info shouldn't make it worse.

However, this will probably break obscure cases that normally only @compnerd and @joerg pick up. I, personally, don't have any opinion on how much this makes sense or not. :/

I'm with @fhahn that we should make ARM and AArch64 as common as possible, not only in code for the target parser but also in behaviour for the front-end, mostly because ARMv8 spans both and we want to avoid surprises.


Repository:
  rC Clang

https://reviews.llvm.org/D45240





More information about the cfe-commits mailing list