[Openmp-commits] [EXTERNAL] [PATCH] D139139: Enable kmpc_atomic functions for arm64

Bran Hagger via Openmp-commits openmp-commits at lists.llvm.org
Mon Dec 5 11:07:54 PST 2022

It produces warnings when built with MSVC, but I think you're right that this is probably okay. The warnings can be suppressed in the command line arguments for the build. It looks like they already occur in a few other places, as we've already suppressed them in our build pipeline.

Thank you,
From: Martin Storsjö via Phabricator <reviews at reviews.llvm.org>
Sent: Sunday, December 4, 2022 1:17 PM
To: Bran Hagger <brhagger at microsoft.com>; jdoerfert at anl.gov <jdoerfert at anl.gov>; martin at martin.st <martin at martin.st>
Cc: zhang.guansong at gmail.com <zhang.guansong at gmail.com>; xw111luoye at gmail.com <xw111luoye at gmail.com>; vigbalas at amd.com <vigbalas at amd.com>; sunil.shrestha at hpe.com <sunil.shrestha at hpe.com>; soutomar at amd.com <soutomar at amd.com>; sara.royuela at bsc.es <sara.royuela at bsc.es>; Saiyedul.Islam at amd.com <Saiyedul.Islam at amd.com>; ron.lieberman at amd.com <ron.lieberman at amd.com>; ravi.narayanaswamy at intel.com <ravi.narayanaswamy at intel.com>; misono.tomohiro at fujitsu.com <misono.tomohiro at fujitsu.com>; michael.hliao at gmail.com <michael.hliao at gmail.com>; lin32 at llnl.gov <lin32 at llnl.gov>; liao6 at llnl.gov <liao6 at llnl.gov>; kevin.sala at bsc.es <kevin.sala at bsc.es>; jinsong.ji at intel.com <jinsong.ji at intel.com>; jdenny.ornl at gmail.com <jdenny.ornl at gmail.com>; jatin.bhateja at gmail.com <jatin.bhateja at gmail.com>; greg63706 at gmail.com <greg63706 at gmail.com>; deepak.eachempati at hpe.com <deepak.eachempati at hpe.com>; david.green at arm.com <david.green at arm.com>; chichun.chen at hpe.com <chichun.chen at hpe.com>; bhuvanendra.kumarn at amd.com <bhuvanendra.kumarn at amd.com>; balaji-sankar-naga-sai-sandeep.kosuri at hpe.com <balaji-sankar-naga-sai-sandeep.kosuri at hpe.com>; abidmuslim at gmail.com <abidmuslim at gmail.com>; a.bataev at hotmail.com <a.bataev at hotmail.com>; nataliaglagoleva at gmail.com <nataliaglagoleva at gmail.com>; stefomeister at gmail.com <stefomeister at gmail.com>; openmp-commits at lists.llvm.org <openmp-commits at lists.llvm.org>; kristof.beyls at arm.com <kristof.beyls at arm.com>; phoebe.wang at intel.com <phoebe.wang at intel.com>
Subject: [EXTERNAL] [PATCH] D139139: Enable kmpc_atomic functions for arm64

mstorsjo added a comment.

Thanks, this version builds successfully for me - but I presume this configuration produces warnings when built with MSVC?

(Btw, when uploading diffs, keep in mind that Phabricator works best if you include extra context, e.g. `git diff -U999`.)

  rG LLVM Github Monorepo



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20221205/4e7bf544/attachment-0001.html>

More information about the Openmp-commits mailing list