================
@@ -5647,20 +5639,15 @@ Value *CodeGenFunction::EmitAArch64BuiltinExpr(unsigned 
BuiltinID,
   }
 
   if (BuiltinID == clang::AArch64::BI_ReadStatusReg ||
-      BuiltinID == clang::AArch64::BI_WriteStatusReg ||
-      BuiltinID == clang::AArch64::BI__sys) {
+      BuiltinID == clang::AArch64::BI_WriteStatusReg) {
     LLVMContext &Context = CGM.getLLVMContext();
 
     unsigned SysReg =
       E->getArg(0)->EvaluateKnownConstInt(getContext()).getZExtValue();
 
     std::string SysRegStr;
-    unsigned SysRegOp0 = (BuiltinID == clang::AArch64::BI_ReadStatusReg ||
-                          BuiltinID == clang::AArch64::BI_WriteStatusReg)
-                             ? ((1 << 1) | ((SysReg >> 14) & 1))
-                             : 1;
     llvm::raw_string_ostream(SysRegStr)
-        << SysRegOp0 << ":" << ((SysReg >> 11) & 7) << ":"
+        << (0b10 | SysReg >> 14) << ":" << ((SysReg >> 11) & 7) << ":"
----------------
Lukacma wrote:

I don't think it is necessary. We have Sema checks ensuring the other bits16 
onwards are set to 0.

https://github.com/llvm/llvm-project/pull/187290
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to