[PATCH] D36336: [X86] Add support for __builtin_cpu_init

Eric Christopher via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Aug 27 22:09:01 PDT 2017


echristo accepted this revision.
echristo added a comment.
This revision is now accepted and ready to land.

One inline comment, but go ahead and commit after fixing that up.



================
Comment at: lib/CodeGen/CGBuiltin.cpp:7292
                                            const CallExpr *E) {
+  if (BuiltinID == X86::BI__builtin_cpu_init) {
+    llvm::FunctionType *FTy = llvm::FunctionType::get(VoidTy,
----------------
I realize it makes sense to put this here because it doesn't require the setup, but let's go ahead and just put it in the switch statement below. I don't think avoiding the bit of setup/etc below is worth the inconsistency. :)


https://reviews.llvm.org/D36336





More information about the cfe-commits mailing list