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

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

Reply via email to