[PATCH] D128120: [Clang][VE] Add missing intrinsics

2022-06-20 Thread Erich Focht via Phabricator via cfe-commits 
efocht accepted this revision.
efocht added a comment.
This revision is now accepted and ready to land.

Looks ok for me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128120

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

[PATCH] D157813: [VE][Clang] Change to enable VPU flag by default

2023-08-14 Thread Erich Focht via Phabricator via cfe-commits 
efocht accepted this revision.
efocht added a comment.
This revision is now accepted and ready to land.

Looks good.
The default should be able to use intrinsics, hiding this only makes things 
complicated and gets users confused.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157813

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

[PATCH] D158558: [clang] - Add missing builtin name to AtomicExpr JSON dump

2023-08-25 Thread Erich Focht via Phabricator via cfe-commits 
efocht added a comment.

Can somebody please revert this patch? It kills all builds on 
https://lab.llvm.org/buildbot/#/builders/91 since yesterday. @serge-sans-paille 
?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158558

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