CarolineConcatto marked an inline comment as done.
CarolineConcatto added a comment.

Thank you for your review @dmgreen  and @tschuett.
I rebase the patch, now  VScaleForTuning  is being set.
And I removed support for neoverse-512tvb from Arm 32 bits. 
@paulwalker-arm pointed me that neoverse-512tvb is only supported by AArch64.



================
Comment at: llvm/lib/Support/Host.cpp:216
         .Case("0xd49", "neoverse-n2")
+        .Case("0xd4a", "neoverse-512tvb")
         .Default("generic");
----------------
dmgreen wrote:
> This doesn't sound right - for a fake cpu to work with -mcpu=native.
Good catch, I should have removed that.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112406/new/

https://reviews.llvm.org/D112406

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

Reply via email to