================
@@ -1549,6 +1549,9 @@ LogicalResult 
ModuleTranslation::convertOneFunction(LLVMFuncOp func) {
   if (auto tuneCpu = func.getTuneCpu())
     llvmFunc->addFnAttr("tune-cpu", *tuneCpu);
 
+  if (auto reciprocalEstimates = func.getReciprocalEstimates())
----------------
tarunprabhu wrote:

Just curious. Is `reciprocalEstimates` an `std::optional` here? If it is a 
pointer, perhaps `auto*` would be more appropriate, but if it is an 
`std::optional`, this is fine.

https://github.com/llvm/llvm-project/pull/143418
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to