[PATCH] D155430: [clang][Interp] Implement __arithmethic_fence for floating types

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 31 06:01:16 PDT 2023


aaron.ballman added a subscriber: Izaron.
aaron.ballman added a comment.

Hmmm, something is weird here. The original patch adding the LLVM intrinsic is https://reviews.llvm.org/D99675 and that says: `The invocation of __arithmetic_fence is not a C/C++ constant expression, even if the operands are constant.` yet https://reviews.llvm.org/D136036 changed arithmetic fence to be a constant expression. I think that was a mistake and we should back that bit of the change out. WDYT @Izaron ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155430



More information about the cfe-commits mailing list