[PATCH] D20088: Using AArch64TargetParser in clang

Renato Golin via cfe-commits cfe-commits at lists.llvm.org
Tue May 10 03:43:17 PDT 2016


rengolin added a comment.

In http://reviews.llvm.org/D20088#425589, @rengolin wrote:

> In this case, it may be simpler to just rely on the Triple object that already exists in the driver, or just the fact that this is in getAArch64ArchFeaturesFromMarch() and pass an extra flag to parseArch() forcing it to only recognise 32-bit or 64-bit arches as valid.


Or, better still, when this is class-based and has inheritance, the AArch64TargetParser will only be used for 64-bit targets, so it'll know already what to refuse (ie. anything before v8).


Repository:
  rL LLVM

http://reviews.llvm.org/D20088





More information about the cfe-commits mailing list