hhb added inline comments.

================
Comment at: lib/CodeGen/CGBuiltin.cpp:9013
       Features, llvm::ConstantInt::get(Int32Ty, FeaturesMask));
   return Builder.CreateICmpNE(Bitset, llvm::ConstantInt::get(Int32Ty, 0));
 }
----------------
craig.topper wrote:
> This code looks to be out of date. It's missing the changes from r344832 that 
> added another runtime variable that presumably has the same issue.
By the way, what is the reason for gcc to add a separate __cpu_feature2, but 
not adding an element to __cpu_model.__cpu_features array?


https://reviews.llvm.org/D53850



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to