[llvm] [TargetParser][AArch64] Believe runtime feature detection (PR #95694)

via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 17 10:44:30 PDT 2024


workingjubilee wrote:

g-d forbid, this is actually done? crypto is just inferred to mean something based on aarch64 version? what.

https://github.com/llvm/llvm-project/blob/4447e255a908c4e1a2863374eaee4bc98e773c3d/llvm/lib/TargetParser/AArch64TargetParser.cpp#L184C53-L184C57

let me rephrase my previous statement: you want me to delete the crypto feature because it is incoherent nonsense, and leave it to downstreams to decide if they want to support this grab-bag of whatever, right?

https://github.com/llvm/llvm-project/pull/95694


More information about the llvm-commits mailing list