================
@@ -1902,6 +1885,7 @@ emitTargetArchBuiltinExpr(CIRGenFunction *cgf, unsigned 
builtinID,
   case llvm::Triple::ppc64le:
   case llvm::Triple::r600:
----------------
andykaylor wrote:

```suggestion
  case llvm::Triple::r600:
    return std::nullopt;
```
This shouldn't fall through. Maybe also duplicate the comment from line 1897.

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

Reply via email to