[PATCH] D86229: [X86] Enable constexpr on POPCNT intrinsics (PR31446)

[Matt Arsenault via Phabricator via cfe-commits]

arsenm added inline comments.


================
Comment at: clang/test/CodeGen/popcnt-builtins.cpp:3
+// RUN: %clang_cc1 -std=c++11 -ffreestanding %s -triple=x86_64-apple-darwin -emit-llvm -o - | FileCheck %s
+
+
----------------
RKSimon wrote:
> craig.topper wrote:
> > RKSimon wrote:
> > > arsenm wrote:
> > > > Missing required register target?
> > > sure, oddly we don't appear to do this for most of the x86 builtins codegen tests - I'll see if I can cleanup those soon
> > Do we need a registered target if we're stopping at IR generation?
> Not really - I'm not sure if its good practice or not, some targets seem to do this but x86 not so much.
There's a vague desire to be able to omit the target intrinsics if the target isn't built, which would imply eventually making the clang parts conditionally compiled


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86229/new/

https://reviews.llvm.org/D86229