================
@@ -38,16 +38,22 @@ void BPFTargetInfo::getTargetDefines(const LangOptions 
&Opts,
 
   Builder.defineMacro("__BPF_FEATURE_ADDR_SPACE_CAST");
 
-  if (CPU.empty() || CPU == "generic" || CPU == "v1") {
+  if (CPU == "generic" || CPU == "v1") {
----------------
eddyz87 wrote:

Nitpick: I think it would be more clear to add a line like below:

```
if (CPU.empty)
  CPU = "v3";
```

(and do the same thing in `BPFSubtarget::initSubtargetFeatures`)

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

Reply via email to